-
Notifications
You must be signed in to change notification settings - Fork 3
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
Gradle, Dependencies and workflows Updated #65
Conversation
with: | ||
java-version: 11 | ||
distribution: temurin | ||
java-version: 17 |
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.
can you also update the dockerfile
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.
Sure!
Yes those can be updated. You can find the compatibility matrix for those two here - https://docs.confluent.io/platform/current/installation/versions-interoperability.html
ok -- I think those should be updated as well (and quite important as security is handled through radar-jersey). If you are having issues, please create a new issue so those can be handled too. @mpgxvii can you take a look if you have bandwidth? |
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.
LGTM, thank you.
Hi @yatharthranjan, also created a new issue for incompatible dependencies here. |
Fixes: #54
Hi @yatharthranjan, I haven't updated the
radarCommonsVersion
andradarJerseyVersion
to the most recent release because some project structure changes are breaking the project.I am also not sure about updating the
confluentVersion
andkafkaVersion
. Please let me know if they need to be updated.