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

Drop Java 8 API compatibility, enforce Java 11 API compatibility #1093

Merged
merged 2 commits into from
Aug 8, 2024

Conversation

artur-ciocanu
Copy link
Contributor

Description

This removes Maven Animal Sniffer plugins and ensures that Dapr Java SDK enforces Java 11 API compatibility. This eventually will help us migrate to Java 17, when we are ready.

Issue reference

We strive to have all PR being opened based on an issue, where the problem or feature have been discussed prior to implementation.

Please reference the issue this PR will close: #1092

Checklist

Please make sure you've completed the relevant tasks for this PR, out of the following list:

  • Code compiles correctly
  • Created/updated tests
  • Extended the documentation

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
@artur-ciocanu artur-ciocanu requested review from a team as code owners July 26, 2024 14:47
@artur-ciocanu
Copy link
Contributor Author

@artursouza and @cicoyle do let me know if you think this PR can be merged or if there are valid reason to force Java 8 API compatibility.

@artur-ciocanu
Copy link
Contributor Author

@artursouza and @cicoyle could you please review and let me know if this PR looks good. Thank you!

@artur-ciocanu
Copy link
Contributor Author

@artursouza as discussed over Discord, could you please approve this PR? Thank you.

@artur-ciocanu
Copy link
Contributor Author

artur-ciocanu commented Aug 5, 2024

@artursouza and @cicoyle could you please review, approve and merge. Thank you!

BTW the build 🟢 !

Copy link
Contributor

@cicoyle cicoyle left a comment

Choose a reason for hiding this comment

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

This looks ok to me given people who want java 8 still will use earlier versions of the sdk.

@artursouza artursouza merged commit 15bbb39 into dapr:master Aug 8, 2024
7 checks passed
@artur-ciocanu artur-ciocanu deleted the gh-1092 branch August 8, 2024 19:32
salaboy pushed a commit to salaboy/java-sdk that referenced this pull request Aug 9, 2024
…r#1093)

Signed-off-by: Artur Ciocanu <ciocanu@adobe.com>
Co-authored-by: Artur Ciocanu <ciocanu@adobe.com>
Signed-off-by: salaboy <Salaboy@gmail.com>
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.

Drop Java 8 API enforcement
3 participants