Switch to GitLab CI

Created on 24 April 2024, 8 months ago
Updated 26 June 2024, 6 months ago

Automated testing on Drupal's infrastructure is being replaced by GitLab CI. I suggest the following steps:

  1. Install the GitLab CI template
  2. Fix any low-hanging fruit
  3. Disable any jobs that are not currently passing
  4. Commit
  5. Create follow-up issues for disabled jobs
  6. Switch the automated tests over to GitLab
📌 Task
Status

Fixed

Version

1.0

Component

Code

Created by

🇦🇺Australia darvanen Sydney, Australia

Live updates comments and jobs are added and updated live.
Sign in to follow issues

Merge Requests

Comments & Activities

  • Issue created by @darvanen
  • Merge request !11GitLab CI → (Merged) created by darvanen
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7
    last update 8 months ago
    12 pass
  • Pipeline finished with Success
    8 months ago
    Total: 236s
    #155206
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7
    last update 8 months ago
    12 pass
  • Pipeline finished with Success
    8 months ago
    Total: 184s
    #155217
  • Open in Jenkins → Open on Drupal.org →
    Core: 9.5.x + Environment: PHP 8.0 & MySQL 5.7
    last update 8 months ago
    12 pass
  • Pipeline finished with Success
    8 months ago
    Total: 173s
    #155238
  • Issue was unassigned.
  • Status changed to Needs review 8 months ago
  • 🇦🇺Australia darvanen Sydney, Australia

    I've put the template in place, I only enabled the "next minor" test because the module does not yet advertise compatibility with the next major. I didn't bother with "previous major" because Drupal 9 is EOL.

    The few phpcs errors were easily fixed so I did that.

    CSpell fails mostly because of the use of UK rather than US English (the latter being the Drupal standard). Didn't know whether you'd want that being messed with so I disabled that test.

    The next minor phpstan test is failing because of https://www.drupal.org/node/3436275 . I've asked in the #contribute channel in Slack for some advice around that. We could disable the next minor test but that would only work until 10.3 is released, at which point something will need to be done. I'd suggest moving forward with the rest of this issue and making a follow-up for that.

    Not sure why phpunit is failing on the next minor, I think that too needs a follow-up.

    Marking NR for maintainer input/direction.

  • 🇮🇹Italy sardara

    Thanks @darvanen for the issue.
    Regarding cspell, we can fix the labels, they are mostly in comments and tests. Let's do it in a follow-up.
    For phpstan, I guess for now we just ignore it. We will handle it later on.
    Phpunit is probably failing due to HTML structure changes, I agree to move it.

  • Status changed to Fixed 8 months ago
  • 🇦🇺Australia darvanen Sydney, Australia

    Follow ups:

  • Status changed to Needs work 8 months ago
  • 🇦🇺Australia darvanen Sydney, Australia

    Still one step to go here - remove the old Drupal CI on this page: https://www.drupal.org/node/3184173/qa

    I'm afraid I can't do that one, maintainer access only.

  • Status changed to Fixed 6 months ago
  • 🇮🇹Italy sardara

    Old CI disabled. Thanks @darvanen!

  • Status changed to Fixed 6 months ago
Production build 0.71.5 2024