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

mirror.yaml: pass PIPELINE_NAME #20

Merged
merged 1 commit into from
May 8, 2024
Merged

mirror.yaml: pass PIPELINE_NAME #20

merged 1 commit into from
May 8, 2024

Conversation

veprbl
Copy link
Member

@veprbl veprbl commented May 6, 2024

Should be a quality-of-life improvement.

@veprbl veprbl force-pushed the pr/pipeline_name branch 2 times, most recently from 052c833 to b1c03fb Compare May 6, 2024 23:24
@veprbl veprbl force-pushed the pr/pipeline_name branch from b1c03fb to 7a76a02 Compare May 6, 2024 23:25
@veprbl
Copy link
Member Author

veprbl commented May 6, 2024

@wdconinc check this out:

image

@@ -8,6 +8,7 @@ variables:
GITHUB_REPOSITORY: ''

workflow:
name: '$PIPELINE_NAME'
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What if triggered from eic_container and $PIPELINE_NAME not set?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Then the name setting is ignored, I believe,
https://docs.gitlab.com/ee/ci/yaml/#workflowname

@veprbl veprbl merged commit 7be4db4 into master May 8, 2024
4 checks passed
@veprbl veprbl deleted the pr/pipeline_name branch May 8, 2024 01:31
veprbl added a commit to eic/epic that referenced this pull request May 8, 2024
github-merge-queue bot pushed a commit to eic/epic that referenced this pull request May 9, 2024
github-merge-queue bot pushed a commit to eic/epic that referenced this pull request May 10, 2024
veprbl added a commit to eic/epic that referenced this pull request May 23, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants