Problem/Motivation

The gitlab_templates project can run tests against multiple downstream projects when an MR is made to ensure that it works on those projects. This feature is really useful to test changes before merging.

For that to happen, the following lines need to stay that way in the .gitlab-ci.yml file:

  - project: $_GITLAB_TEMPLATES_REPO
    ref: $_GITLAB_TEMPLATES_REF

This project has ref: main, to get the latest changes as they are merged, but that means that we can no longer test downstream pipelines (thanks to @jonathan1055 for realizing this!).

Proposed resolution

Either change it back to the recommended settings or just inform that the project will stay on main, which is totally fine, so that we can select another project as a downstream candidate.

I will create the MR for the first option, but feel free to not merge and close the issue if you want to stay in the "main" ref.

Issue fork decoupled_pages-3422869

Command icon Show commands

Start within a Git clone of the project using the version control instructions.

Or, if you do not have SSH keys set up on git.drupalcode.org:

About issue forks

Comments

fjgarlin created an issue. See original summary.

fjgarlin’s picture

fjgarlin commented
Title: Revert main to recommended to keep downstream jobs working as expected » Revert main to recommended ref to keep downstream jobs working as expected

fjgarlin opened merge request !4

fjgarlin’s picture

fjgarlin commented
Status: Active » Needs review

MR is ready for review.

fjgarlin’s picture

fjgarlin commented
Issue summary: View changes

@jonathan1055 found this out yesterday when we were testing changes on an ongoing issue.

Wim Leers credited jonathan1055.

wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Needs review » Reviewed & tested by the community

Oh, oops! 😅

  • Wim Leers committed bbfc9923 on 8.x-1.x authored by fjgarlin 
    Issue #3422869 by fjgarlin, jonathan1055: Revert main to recommended ref...
wim leers’s picture

wim leers
Location Ghent 🇧🇪🇪🇺
commented
Status: Reviewed & tested by the community » Fixed
jonathan1055’s picture

jonathan1055 commented

Oh, oops!

Don't worry. I am pleased you wanted to test with 'main' branch.

Status: Fixed » Closed (fixed)

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