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

Just revamping the gradle build to use up-to-date tooling. #255

Conversation

GuillaumeTaffin
Copy link
Contributor

@GuillaumeTaffin GuillaumeTaffin commented Feb 24, 2024

Just revamping the gradle build to use up-to-date tooling.
None of this changes should change the build logic or output, but will hopefully help maintenance by keeping the build to the latest Gradle's recommendations.

  • Update the gradle wrapper to version 8.6
  • Replace deprecated "version" by "archiveVersion" in the common build config within the jar.manifest.attributes {}
  • Put all dependencies in the libs.versions.toml catalog
  • Apply the unit testing dependencies by default to all subprojects in the common build config
  • Update the checkout and setup-java github actions to the v4
  • Run the CI for all branches

@GuillaumeTaffin GuillaumeTaffin force-pushed the updage-gradle-build branch 4 times, most recently from d2be353 to 60a4c7b Compare February 24, 2024 17:11
None of this changes should change the build logic or output, but will hopefully help maintenance by keeping the build to the latest Gradle's recommendations.

- Update the gradle wrapper to version 8.6
- Replace deprecated "version" by "archiveVersion" in the common build config within the jar.manifest.attributes {}
- Put all dependencies in the libs.versions.toml catalog
- Apply the unit testing dependencies by default to all subprojects in the common build config
- Update the checkout and setup-java github actions to the v4
- Run the CI for all branches
@GuillaumeTaffin
Copy link
Contributor Author

@simonbrowndotje Are you interested in this ? I can do the same kind of update for other repos in the project if you want to have more consistency

@galuszkak
Copy link

@simonbrowndotje do you plan to review this?

@GuillaumeTaffin GuillaumeTaffin closed this by deleting the head repository Jul 27, 2024
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