Skip to content

Conversation

yrodiere
Copy link
Member

@yrodiere yrodiere commented Nov 18, 2024

Upgrade to 1.18 to avoid buggy notifications.
See https://hibernate.zulipchat.com/#narrow/channel/132096-hibernate-user/topic/Jenkins.20Notifications
Full changeset: https://github.com/hibernate/hibernate-jenkins-pipeline-helpers/compare/1.17..1.18

Don't configure the version in Jenkinsfile so that it's easier to upgrade next time.
See https://ci.hibernate.org/manage/configure#global-untrusted-pipeline-libraries to change the (default) version globally for all projects.


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.
For more information on licensing, please check here.


Copy link

Copy link
Member

@marko-bekhta marko-bekhta left a comment

Choose a reason for hiding this comment

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

This looks nice 🎉 🥳 less version updates to think about 😄

Thinking of whether this can be a problem in the future, I could only come up with this case:
If we introduce a breaking change in the helper lib, then we'd need to go back to all (maintained) branches (where the version is not specified) to adjust the Jenkins file (since it doesn't point to a particular version anymore)

@yrodiere
Copy link
Member Author

Thinking of whether this can be a problem in the future, I could only come up with this case: If we introduce a breaking change in the helper lib, then we'd need to go back to all (maintained) branches (where the version is not specified) to adjust the Jenkins file (since it doesn't point to a particular version anymore)

Yep. So let's not do that :)

@marko-bekhta marko-bekhta merged commit cc8cf14 into hibernate:main Nov 19, 2024
8 checks passed
@marko-bekhta
Copy link
Member

let's keep that in mind 😄
Thanks!

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