-
Notifications
You must be signed in to change notification settings - Fork 21
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
Bump default gradle-nexus.publish-plugin
version
#1479
Conversation
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.
Minor comment on the changelog stuff but LGTM otherwise
CHANGELOG.md
Outdated
### Improvements | ||
|
||
- Update `io.github.gradle-nexus.publish-plugin` to 2.0.0 from 1.1.0. | ||
|
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.
If you add this to CHANGELOG_PENDING
it gets taken care of for you as part of the release process.
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.
My kingdom for a consistent changelog process :)
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.
I have some hesitation with respect to this dependency so I hope we tried running it at least once to see it produces the releases and does not require changes from our SDK gen setup (2.0 might be breaking after all) before rolling out here. But otherwise LGTM :)
This is a sanity check to ensure Java publishing still works `io.github.gradle-nexus.publish-plugin` v2. Refs pulumi/pulumi-java#1479.
@t0yv0 of course -- https://github.com/pulumi/pulumi-command/actions/runs/11942352524/job/33292492882 seems there was a flake but it eventually succeeded. |
Release notes.
We don't use this option, so this shouldn't impact us.
Refs https://github.com/pulumi/home/issues/3762.