-
Notifications
You must be signed in to change notification settings - Fork 15
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
[CI] Use correct SNAPSHOT version for Quarkus release branches #673
[CI] Use correct SNAPSHOT version for Quarkus release branches #673
Conversation
8c7ddb2
to
1e3c6b3
Compare
I re-scheduled the CI run since the original was canceled due to a newer Nightly run. |
Failures are due to oracle#8075 / quarkusio/quarkus#37657 |
Weekly CI was not used on this repository because of:
But it run in my fork: https://github.com/zakkak/mandrel/actions/runs/7641569142 and indicates that there is a mistake in my bash conditionals. |
Release branches are now using <version>.999-SNAPSHOT versions See https://github.com/quarkusio/quarkus/blob/3c8c772dda9ed99674c9d8a6cd9cb4635d1ce5a2/build-parent/pom.xml#L8 and https://github.com/quarkusio/quarkus/blob/bc0ca288d32ba2df212d8c4a2783c0bb21ca99a1/build-parent/pom.xml#L8 Related to graalvm#501
69e6a5c
to
e9d51eb
Compare
The PR depends on Karm/mandrel-integration-tests#247, without it the tests fail with:
See https://github.com/zakkak/mandrel/actions/runs/7655379970/job/20862301754#step:11:4258 |
The changes work as expected, see https://github.com/zakkak/mandrel/actions/runs/7655379970 now that Karm/mandrel-integration-tests#247 is merged. |
Release branches are now using .999-SNAPSHOT versions
See
https://github.com/quarkusio/quarkus/blob/3c8c772dda9ed99674c9d8a6cd9cb4635d1ce5a2/build-parent/pom.xml#L8
and
https://github.com/quarkusio/quarkus/blob/bc0ca288d32ba2df212d8c4a2783c0bb21ca99a1/build-parent/pom.xml#L8
Related to #501