Problem/Motivation

Never got around to setting up GitLab CI for this project.

Steps to reproduce

Proposed resolution

Remaining tasks

  1. Add .gitlab-ci.yml file
  2. See how the builds do
  3. Either fix easy things or create follow-ups to get happy green pipelines

User interface changes

API changes

Data model changes

Issue fork views_taxonomy_term_name_into_id-3482593

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

dww created an issue. See original summary.

dww opened merge request !4

dww’s picture

dww
we/he/they
commented
Status: Active » Needs review

Trivial MR up to get this started. Let's see how https://git.drupalcode.org/project/views_taxonomy_term_name_into_id/-/pi... goes...

dww’s picture

dww
we/he/they
commented
Status: Needs review » Needs work

Wheee, that pipeline failed on basically every step. 😉

  • cspell
  • phpcs
  • phpstan
  • phpunit (current)
  • phpunit (previous major)

This isn't yet declared compatible with D11, so no surprise that phpunit (current) is failing...

vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Assigned: Unassigned » vladimiraus
vladimiraus’s picture

vladimiraus
Primary language English
Location Brisbane 🇦🇺
commented
Assigned: vladimiraus » Unassigned
Status: Needs work » Needs review
lavanyatalwar’s picture

lavanyatalwar commented

The changes look good to me. Also the pipeline is green now.
Moving it to RTBC.

lavanyatalwar’s picture

lavanyatalwar commented
Status: Needs review » Reviewed & tested by the community
dww’s picture

dww
we/he/they
commented

I reverted all the commits in the MR since #3. They're out of scope here (D11 porting) or totally unnecessary / unhelpful (adding a composer.json file).

I pushed new commits to explicitly test the core versions we're supporting. Also, fixed phpcs, phpstan, cspell.

If the latest pipeline is green on all 3 configurations, I'll merge this.

dww’s picture

dww
we/he/they
commented

I reverted all the commits in the MR since #3. They're out of scope here (D11 porting) or totally unnecessary / unhelpful (adding a composer.json file).

I pushed new commits to explicitly test the core versions we're supporting. Also, fixed phpcs, phpstan, cspell.

If the latest pipeline is green on all 3 configurations, I'll merge this.

  • dww committed 325f6c2a on 8.x-1.x 
    task: #3482593 Enable GitLab CI and cleanup build pipelines

By: @dww

dww closed merge request !4

dww’s picture

dww
we/he/they
commented
Status: Reviewed & tested by the community » Fixed

Now that this issue is closed, review the contribution record.

As a contributor, attribute any organization that helped you, or if you volunteered your own time.

Maintainers, credit people who helped resolve this issue.

Status: Fixed » Closed (fixed)

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