Skip to content

Conversation

diogob003
Copy link
Contributor

@diogob003 diogob003 commented Aug 12, 2025

  • Move plugins from legacy buildscript block to plugins
  • Centralize repository management in settings.gradle.kts (default for new A.S. projects)
  • Use alias() to import plugins info from version catalog libs.versions.toml
  • Upgrade wrapper 8.13 -> 9.1.0 gradle wrapper --gradle-version 9.1.0

Copy link
Collaborator

@fire-light42 fire-light42 left a comment

Choose a reason for hiding this comment

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

@C10udburst Can you take a closer look at this pull request, as I am not up to date with the build process.

@diogob003 diogob003 force-pushed the security-libs-ver-fd branch 2 times, most recently from af4f626 to 5bc9584 Compare August 12, 2025 16:06
@Dexbuddy
Copy link

Why isn't this merged yet?

@fire-light42
Copy link
Collaborator

Why isn't this merged yet?

There is a lot of changes, and no one else have looked at it. I do not have the extensive knowledge about the build system to merge this in good faith.

@diogob003
Copy link
Contributor Author

@fire-light42 I can try to split it even further. I'll create a new pr that only modifies libs.versions.toml, and let this pull request for changes on the build scripts and Gradle Wrapper.
I did not make any major changes to the build scripts, as most of it was autogenerated when I upgraded the Gradle wrapper version.

* Upgrade `gradle wrapper --gradle-version 9.1.0`
* Move plugins from legacy buildscript block to plugins
* Centralize repository management in settings.gradle.kts (default for new A.S. projects)
* Use alias() to import plugins info from version catalog libs.versions.toml
@diogob003 diogob003 force-pushed the security-libs-ver-fd branch from 5bc9584 to fd04485 Compare October 19, 2025 22:41
@diogob003 diogob003 changed the title Upgrade libraries version and gradle wrapper Upgrade gradle wrapper Oct 19, 2025
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.

4 participants