Contrib.social

Gitlab CI: Fix validation errors reported by jobs CSPELL, PHPCS and PHPSTAN

Open on Drupal.org →
Created on 23 May 2025, 10 days ago

Problem/Motivation

This is a follow-up to 📌 Automated testing: Configure GitLab CI Active , where the bare minimum configuration was added to enable automated testing on Gitlab CI.

Validation jobs now report errors in build pipelines, see latest build:
https://git.drupalcode.org/project/config_perms/-/pipelines/504276

Steps to reproduce

Run build pipeline at:
https://git.drupalcode.org/project/config_perms/-/pipelines/new

Or each of the jobs listed above separately locally.

Proposed resolution

Fix all validation errors for CSPELL, PHPCS and PHPSTAN.

This was addressed already in a separate merge request but considered out-of-scope, so it was not merged yet:
https://git.drupalcode.org/project/config_perms/-/merge_requests/6

Move over the work that was done previously to a new merge request for this issue specifically.

📌 Task
Status

Active

Version

2.0

Component

Code

Created by

🇫🇷France dydave

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

Merge Requests

Comments & Activities

  • Issue created by @dydave
  • Pipeline finished with Success
    10 days ago
    Total: 169s
    #504801
  • Merge request !7Issue #3516579 by dydave: Automated testing on GitLab CI: Added initial... → (Open) created by dydave
  • Pipeline finished with Success
    10 days ago
    Total: 138s
    #504806
  • Pipeline finished with Canceled
    10 days ago
    Total: 297s
    #504812
  • Pipeline finished with Success
    10 days ago
    Total: 240s
    #504817
  • Comment 10 days ago →
    🇫🇷France dydave

    Quick follow-up on this issue:

    • Created a new issue fork with this issue.
    • Merged the branch 3516579-configure-gitlabci locally and pushed to this issue fork.
    • Added extra commit to broaden Tests coverage and require all jobs to pass.
    • Created merge request MR !7 above at #2.

     
    Since all the jobs are now passing 🟢, moving issue to Needs review.
     
    Please note: The changes to require jobs and broaden Tests coverage (GitLab CI config file) are completely at the appreciation and preference of the maintainers.
    Feel free to adjust as you would see fit 🙂
    Whatever works best for you!
     
    Any feedback, comments or reviews would be greatly appreciated.
    Thanks in advance!

contrib.social Blog FAQ Discussions
Production build 0.71.5 2024