Add test coverage for stylelint job

We need this now, because 📌 Exclude node_modules folder from artifacts Active makes changes to the Stylelint job, but we have no downstream test coverage of this.

jonathan1055 → changed the visibility of the branch 3513408-test-coverage-for-stylelint to hidden.

Tested via UI with new variable BEFORE_SCRIPT_ACTIONS set to stylelint-fail, and the job fails as required.
https://git.drupalcode.org/issue/gitlab_templates_downstream-3513408/-/j...

The MR looks good and simple. RTBC.

Thanks. Merged in d10-plus and cherry-picked into d10-theme and d9-basic. Both cherry-picks were clean and did not require any resolving of conflicts. Keeping the .gitlab-ci.yml file aligned where applicable means we make it easier for ourselves going forward merging things like this. But I didn't add this to main or d10-profile as the .css file is not so relevent on those branches. I presume that's OK?

100% ok. We will try to keep them aligned (by cherry picking) when it makes sense. If it doesn't make sense, it's better to keep things clean than add extra clutter.

I've created MR27 so that we have permanent test coverage for the non-default _COMPOSER_YARN_INSTALL: 0. But it won't have any effect until 📌 Exclude node_modules folder from artifacts Active is merged, so maybe do that one first.

I am merging the other one now. This is RTBC.

Thanks. I re-ran the MR27 but of course this project does not use 'main' it has to use

  - project: $_GITLAB_TEMPLATES_REPO
    ref: $_GITLAB_TEMPLATES_REF

to enable it to function as desired when used in downstream pipelines. So I ran a pipeline via the issue fork UI setting _gitlab_templates_ref and _curl_templates_ref to 'main' and then we get the newly committed changes, and it works as required
https://git.drupalcode.org/issue/gitlab_templates_downstream-3513408/-/p...

Merged and fixed.

Automatically closed - issue fixed for 2 weeks with no activity.