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: handle legacy stage name differences #4733

Merged
merged 4 commits into from
Jan 10, 2024

Conversation

Googlom
Copy link
Member

@Googlom Googlom commented Dec 21, 2023

As stage names differ between orchestrators and in order to use the same stage conditions file, we need additional logic to handle different names.

@Googlom Googlom requested a review from a team as a code owner December 21, 2023 06:07
Copy link
Member

@CCFenner CCFenner left a comment

Choose a reason for hiding this comment

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

👍

@CCFenner CCFenner enabled auto-merge (squash) January 9, 2024 09:25
@CCFenner CCFenner disabled auto-merge January 9, 2024 09:25
@CCFenner
Copy link
Member

CCFenner commented Jan 9, 2024

/it-go

Copy link

Quality Gate Passed Quality Gate passed

Kudos, no new issues were introduced!

0 New issues
0 Security Hotspots
No data about Coverage
No data about Duplication

See analysis details on SonarCloud

@CCFenner CCFenner changed the title Stage name difference handling fix: handle legacy stage name differences Jan 10, 2024
@CCFenner CCFenner merged commit 2b2c441 into master Jan 10, 2024
11 checks passed
@CCFenner CCFenner deleted the gulom/stagename_difference_handler branch January 10, 2024 09:39
maxatsap pushed a commit to maxatsap/jenkins-library that referenced this pull request Jul 23, 2024
* add name difference handler function

* add conditions for setting keys

---------

Co-authored-by: Gulom Alimov <[email protected]>
Co-authored-by: Christopher Fenner <[email protected]>
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