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

[#44] Upgrading project build #45

Merged
merged 1 commit into from
Nov 27, 2023
Merged

Conversation

kostobog
Copy link
Collaborator

  • upgrade gradle to gradle 8.4
  • upgrade to java 11
  • upgrade jopa to latest 1.1.4
  • upgrade jopa-spring-transaction to latest 0.2.0
  • upgrade jb4jsonld-jackson to latest 0.13.2

upgrade build plugins

  • upgrade plugin org.springframework.boot 2.7.17 latest version supporting java 11
  • upgrade io.spring.dependency-management to latest 1.1.3
  • upgrade io.freefair.aspectj.post-compile-weaving to latest 8.4

The build outputs a warning. Cannot find usage in our codebase or in dependencies.

[warning] Found @DeclareAnnotation while current release does not support it (see 'org.aspectj.weaver.bcel.AtAjAttributes')

- upgrade gradle to gradle 8.4
- upgrade to java 11
- upgrade jopa to latest 1.1.4
- upgrade jopa-spring-transaction to latest 0.2.0
- upgrade jb4jsonld-jackson to latest 0.13.2

upgrade build plugins
- upgrade plugin org.springframework.boot 2.7.17 latest version supporting java 11
- upgrade io.spring.dependency-management to latest 1.1.3
- upgrade io.freefair.aspectj.post-compile-weaving to latest 8.4
@kostobog kostobog requested a review from blcham October 31, 2023 10:31
@blcham
Copy link
Contributor

blcham commented Oct 31, 2023

@kostobog do not know what to do with warning :(

Based on stackoverflow it seem that this method throws the exception, so maybe you can debug it to find out what it has been processing before it throwed the error

@kostobog
Copy link
Collaborator Author

This pull request will be closed as the warning is not a problem in the build and the application.
There is a separate issue for this warning #46.

@kostobog kostobog merged commit 489d921 into main Nov 27, 2023
1 check passed
@blcham blcham deleted the upgrade/issue-44-upgrade-jopa branch January 3, 2024 16:30
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.

2 participants