Skip to content
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

Update byte-buddy to 1.10.20 #9

Merged
merged 1 commit into from
May 10, 2021
Merged

Update byte-buddy to 1.10.20 #9

merged 1 commit into from
May 10, 2021

Conversation

protocol7
Copy link
Collaborator

For Java 15 support, and better integration with Mockito.

For Java 15 support, and better integration with Mockito.
@protocol7 protocol7 requested review from mattnworb and spkrka May 10, 2021 11:37
Copy link

@rydenius rydenius left a comment

Choose a reason for hiding this comment

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

Thank you. Linking https://spotify.slack.com/archives/CM5CTDU07/p1620641145093300 here too for additional context.

@@ -17,7 +17,7 @@ jobs:
strategy:
matrix:
# test against latest update of each major Java version, as well as specific updates of LTS versions:
java: [ 8, 11, 13 ]
java: [ 8, 11, 13, 15 ]

Choose a reason for hiding this comment

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

According to the comment above I would have expected 15 to replace 13 rather than getting appended? But this works too of course.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Don't really have a fully thought-out strategy here, but went with both 13 and 15 as they are MTS releases on some distros (e.g. Azul https://www.azul.com/products/azul-support-roadmap/). Once 17 gets released, I would imagine we would remove 13 and 15.

@protocol7 protocol7 merged commit 2c5110a into main May 10, 2021
@protocol7 protocol7 deleted the byte-buddy-1.10.20 branch May 10, 2021 12:09
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.

4 participants