-
Notifications
You must be signed in to change notification settings - Fork 594
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
mavenBuild: add the possibility to define stash groups need to be stashed/unstashed #4573
Conversation
/it-go |
Where was this unstashed before? Is there something we need to remove? |
@CCFenner No, nothing needs to be removed. |
@vstarostin is this PR going to be merged today? |
/it-go |
Kudos, SonarCloud Quality Gate passed! |
@CCFenner, @anilkeshav27 FYI: I removed the default |
…be stashed/unstashed (SAP#4573) * Add stash param to mavenBuild * go generate * Delete default stash group --------- Co-authored-by: Anil Keshav <[email protected]>
…be stashed/unstashed (#4573) * Add stash param to mavenBuild * go generate * Delete default stash group --------- Co-authored-by: Anil Keshav <[email protected]>
…be stashed/unstashed (SAP#4573) * Add stash param to mavenBuild * go generate * Delete default stash group --------- Co-authored-by: Anil Keshav <[email protected]>
Changes
This PR adds the possibility of defining stash groups that need to be unstashed into container.
The default value is 'source' to keep existing behavior and not break existing pipelines.We are not adding any default stash group to not change existing behavior and not break existing pipelines.To set custom stash groups need to be unstashed into container, please set:
This PR is pretty much the same as #2271 (#2285) for npmExecuteScripts step.