Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix: replace deprecated gitlab ci var #1404

Merged
merged 5 commits into from Jul 26, 2023
Merged

Conversation

nipierre
Copy link
Contributor

GitLab 16.x deprecated several variables including CI_BUILD_REF_NAME. In order to be exhaustive, two variables are replacing it: CI_COMMIT_BRANCH (prefered) and CI_COMMIT_REF_NAME.

Issue: #1403

I put the two variables in order to cover most of the cases one can think of in terms of pipeline trigger.

@dacbd

GitLab 16.x deprecated several variables including CI_BUILD_REF_NAME.
In order to be exhaustive, two variables are replacing it:
CI_COMMIT_BRANCH (prefered) and CI_COMMIT_REF_NAME.

Issue: iterative#1403
@nipierre nipierre temporarily deployed to external July 20, 2023 16:12 — with GitHub Actions Inactive
src/drivers/gitlab.js Outdated Show resolved Hide resolved
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd temporarily deployed to external July 20, 2023 16:56 — with GitHub Actions Inactive
@dacbd dacbd added bug Something isn't working cml-pr Subcommand ci-gitlab labels Jul 20, 2023
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd temporarily deployed to external July 20, 2023 17:18 — with GitHub Actions Inactive
@nipierre nipierre temporarily deployed to external July 21, 2023 09:14 — with GitHub Actions Inactive
@nipierre
Copy link
Contributor Author

@dacbd I fixed the credentials build missing argument.

@dacbd
Copy link
Contributor

dacbd commented Jul 26, 2023

@nipierre Thanks, I'm addressing a container issue that is blocking some of our automation, my plan is to try and have this merged and released by the end of the week for you.

Thanks again for this patch!

@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd temporarily deployed to external July 26, 2023 19:03 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd temporarily deployed to external July 26, 2023 20:39 — with GitHub Actions Inactive
@github-actions
Copy link
Contributor

Test Comment

@github-actions
Copy link
Contributor

Test Comment

@dacbd dacbd merged commit f050c11 into iterative:master Jul 26, 2023
27 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working ci-gitlab cml-pr Subcommand
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants