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.

Sign up for GitHub

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

Jump to bottom

Update plugin build and dependencies #103

Closed
wants to merge 1 commit into from
Closed

Update plugin build and dependencies #103

wants to merge 1 commit into from

Conversation

cstamas
Copy link
Member

@cstamas cstamas commented Oct 8, 2022

Update build and make it build on Java 17 as well.

Depends on takari/takari-pom#9

High level changes:

  • updated wrapper to latest
  • uses parent 50 from Update Takari parent POM takari-pom#9
  • remove/adjust prerequisites (needed only for plugin, parent POM not, etc)
  • get rid of animal sniffer as compiler release flag is used instead
  • update plugin dependencies
  • update sources where depedendency introduced change
  • one IT changed: CompileAnnotationProcessingTest as it used source 1.6, that is unsupported by Java 17. Changed to release 7

With these changes the build works okay on Java 17 as well.

@cstamas cstamas self-assigned this Oct 8, 2022
@ajayk ajayk mentioned this pull request Oct 22, 2022
Closed
@cstamas
Copy link
Member Author

cstamas commented Oct 26, 2022

This requires POM v50 release w/ merged changes https://github.com/takari/takari-pom

@cstamas cstamas closed this Nov 5, 2022
@cstamas cstamas deleted the update-plugin branch November 5, 2022 21:34
@cstamas
Copy link
Member Author

cstamas commented Nov 5, 2022

Superseded by #105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees

@cstamas cstamas

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@cstamas