KEMBAR78
Update dependencies junit, mockito, bytebuddy, jcstress, spotless... by chemicL · Pull Request #3218 · reactor/reactor-core · GitHub
Skip to content

Conversation

@chemicL
Copy link
Member

@chemicL chemicL commented Oct 7, 2022

This commit updates the following dependencies:

It also updates the following build dependencies:

And also updates the following test dependencies:

This commit updates the following dependencies:

- bytebuddy 1.12.14 -> 1.12.17, supersedes and closes #3186

It also updates the following build dependencies:

- plugin artifactory 4.29.0 -> 4.29.1, supersedes and closes #3207
- plugin jcstress 0.8.13 -> 0.8.14, supersedes and closes #3171
- plugin download 5.1.2 -> 5.2.1, supersedes and closes #3188
- plugin spotless 6.10.0 -> 6.11.0, supersedes and closes #3193

And also updates the following test dependencies:

- JUnit 5.9.0 -> 5.9.1, supersedes and closes #3186
- Mockito 4.7.0 -> 4.8.0, supersedes and closes #3185
@chemicL chemicL added the type/dependency-upgrade A dependency upgrade (possibly via bot) label Oct 7, 2022
@chemicL chemicL added this to the 3.4.24 milestone Oct 7, 2022
@chemicL chemicL self-assigned this Oct 7, 2022
@chemicL chemicL requested a review from a team as a code owner October 7, 2022 12:53
//the plugin feature only works with the -original jar !!
//otherwise implemented Plugin interface is the shaded one
classpath files("@AGENT@")
//version 1.12.13 cannot be found on Gradle Plugin Repository, this syntax allows looking it up on MavenCentral
Copy link
Contributor

Choose a reason for hiding this comment

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

from what I can see, this is still needed for 1.12.17

Copy link
Member Author

Choose a reason for hiding this comment

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

Thanks for noticing. I noticed some commits in byte buddy regarding publishing and thought things have improved. Unfortunately, 1.12.17 is not present in the repo, while 1.12.16 is there 🤷 . I reverted the workaround, just bumped the version.

@chemicL chemicL requested a review from simonbasle October 7, 2022 13:57
@chemicL chemicL merged commit 2735ba1 into 3.4.x Oct 10, 2022
@reactorbot
Copy link

@chemicL this PR seems to have been merged on a maintenance branch, please ensure the change is merge-forwarded to intermediate maintenance branches and up to main 🙇

@chemicL chemicL deleted the update-dependencies branch October 10, 2022 07:09
chemicL pushed a commit that referenced this pull request Oct 10, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type/dependency-upgrade A dependency upgrade (possibly via bot)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants