- Issue created by @catch
- 🇬🇧United Kingdom longwave UK
The initial GitLab CI commit only enabled pipelines for MRs, so repurposing this issue to add pipelines on push and also manually via the web UI.
- 🇬🇧United Kingdom longwave UK
Added three extra triggers to the default PHP 8.2/MySQL 8 job:
push
for branch pushesschedule
so we can replicate DrupalCI scheduled pipelinesweb
so maintainers can manually trigger runs if needed
- Status changed to Needs review
about 1 year ago 10:03am 13 September 2023 - last update
about 1 year ago 30,150 pass - @longwave opened merge request.
- last update
about 1 year ago 30,150 pass - Status changed to RTBC
about 1 year ago 10:17am 13 September 2023 - 🇪🇸Spain fjgarlin
The MR changes look good to me. It's targeting just the "project" namespace for other manual (web), schedule or push, which makes sense as we don't want/need that in forks.
RTBC.
- Status changed to Active
about 1 year ago 10:23am 13 September 2023 - 🇬🇧United Kingdom catch
Committed/pushed to 11.x, leaving this open for more branch test fun.
- last update
about 1 year ago 30,150 pass - last update
about 1 year ago 30,150 pass - Status changed to Needs review
about 1 year ago 11:03am 13 September 2023 - 🇬🇧United Kingdom longwave UK
CI_MERGE_REQUEST_TARGET_BRANCH_NAME is only set for MRs; CI_COMMIT_BRANCH is only set for branch pushes, so we can concatenate the variables and get the correct value where we need it.
The cspell job tries to look for the set of modified files, which doesn't make sense in a branch run, so we need to figure out what we should do here - a full run of cspell?
- last update
about 1 year ago 30,150 pass - Status changed to Fixed
about 1 year ago 4:06pm 13 September 2023 - 🇬🇧United Kingdom longwave UK
Think this can be marked as fixed now and matrix jobs and cspell have their own individual followups.
Automatically closed - issue fixed for 2 weeks with no activity.