Skip to content

Conversation

wadoon
Copy link
Member

@wadoon wadoon commented Feb 15, 2025

The best practices in Gradle have been moved further. This PR tries to follow.

It introduced:

  • One common libs.versions.toml file for a global handling of dependencies.
    This dependency management tries to avoid using the same Maven coordinates in various places.
    A different solution would be using a BOM.

  • It introduces java-convention plugin, which assimilates the old subproject { ... } configuration

  • Build scripts are transferred to kts (Kotlin) for more type safety.

@wadoon wadoon self-assigned this Feb 15, 2025
@wadoon wadoon changed the title Making the build script fit for the future. Cleanup: Making the build script fit for the future. Feb 15, 2025
@wadoon wadoon added the 🛠 Maintenance Code quality and related things w/o functional changes label Feb 15, 2025
@wadoon wadoon added this to the v2.14.0 milestone Feb 15, 2025
@wadoon wadoon force-pushed the weigl/cleanup/gradle branch 2 times, most recently from a8bb94b to e333703 Compare February 25, 2025 17:46
@wadoon wadoon force-pushed the weigl/cleanup/gradle branch from e333703 to e4b7aa2 Compare March 16, 2025 00:58
@wadoon wadoon force-pushed the weigl/cleanup/gradle branch from e4b7aa2 to 59ae0fb Compare June 8, 2025 14:05
@wadoon wadoon force-pushed the weigl/cleanup/gradle branch from 59ae0fb to 5b61e43 Compare August 16, 2025 05:40
@wadoon wadoon requested a review from unp1 August 18, 2025 07:31
@wadoon
Copy link
Member Author

wadoon commented Aug 18, 2025

In general, this branch is fit, but the checkerFramework does work quite well with pre-compiled build scripts.

Cannot mutate the dependencies of configuration ':recoder:checkerFramework' after the configuration was consumed as a variant. After a configuration has been observed, it should not be modified.

@wadoon wadoon force-pushed the weigl/cleanup/gradle branch from 51a878a to 6af6080 Compare August 23, 2025 13:22
@wadoon wadoon force-pushed the weigl/cleanup/gradle branch from 6af6080 to 95539ae Compare September 7, 2025 22:39
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🛠 Maintenance Code quality and related things w/o functional changes
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant