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.

Sign up for GitHub

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

Jump to bottom

Fix: Changed pipeline variable name #681

Merged
merged 2 commits into from
Aug 27, 2024
Merged

Fix: Changed pipeline variable name #681

merged 2 commits into from
Aug 27, 2024

Conversation

larfeq
Copy link
Collaborator

@larfeq larfeq commented Aug 26, 2024 •
edited
Loading

  • New feature
  • Bug fix
  • High impact

Work description:

The template used the wrong variable name.

Expected result: Pipeline runs again

Link to workitem: AB#55752

Test description:

  • Can be tested
  • Automatic tests created / updated
  • Local tests are passing

CI pipeline runs after update

Checklist:

  • Considered automated tests
  • Considered updating specification / documentation
  • Considered work items
  • Considered security
  • Performed developer testing
  • Checklist finalized / ready for review

@larfeq larfeq requested a review from HansDahle August 26, 2024 08:27
@larfeq larfeq marked this pull request as ready for review August 26, 2024 08:28
@larfeq larfeq changed the title Fix: Changed variable name Fix: Changed pipeline variable name Aug 27, 2024
@larfeq larfeq merged commit 79309c2 into master Aug 27, 2024
16 checks passed
@larfeq larfeq deleted the fix/key-rotation/fixed-pipeline-variable-name branch August 27, 2024 08:40
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@HansDahle HansDahle HansDahle approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@larfeq @HansDahle