-
Notifications
You must be signed in to change notification settings - Fork 547
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
Cleanup and add version to preview builds #3894
Conversation
Your Pull Request was automatically labelled as: "🧹 Chores" |
6f246ea
to
ee626e4
Compare
bit out of scope but can we add the commit to the preview message as signature so we know if it has changed |
973b4a2
to
366b870
Compare
dfb863b
to
69dbecf
Compare
@WalshyDev can you fire a preview build so we know how it looks? |
69dbecf
to
f3f6319
Compare
Nope, it runs on |
ugh |
yeah trying to make this actually run as a test is annoying, in theory it'll work now but I'm sure another error will happen |
ddf460b
to
16871dd
Compare
yeah i give up, we'll just test once merged |
16871dd
to
cd699ce
Compare
cd699ce
to
c0e0b64
Compare
Kudos, SonarCloud Quality Gate passed! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Again, just trusting you know your stuff here xD <3
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Blob goes beep boop
Changes the version in preview builds from
4.9-UNOFFICIAL
toPreview Build #<pr-num>-<hash>
Just a nice way to see what preview build we're running, check it's latest, check it's the right pr, etc.
Also does some cleanup, we no longer do preview stuff in the "Java CI" workflow and try to work around it firing on both push + pr. There's now just a PR workflow