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

Gradle Separation of Concerns #74

Draft
wants to merge 15 commits into
base: main
Choose a base branch
from

Conversation

kaibernhard
Copy link
Contributor

No description provided.

@kaibernhard kaibernhard marked this pull request as draft August 15, 2023 09:36
@kaibernhard kaibernhard force-pushed the gradle-separation-of-concerns branch 2 times, most recently from de3507f to 2fcfdd4 Compare September 1, 2023 08:58
Seperation of concerns:

* define dependency versions in gradle/libs.versions.toml
* define where to get dependencies in settings.gradle.kts
Sets up an integration test suite and configures code coverage for
aggregated and per-suite test reports.

The `sonar task now depends on `check` and `check` depends on the coverage
reports to avoid direct interactions between different convention plugins.
This allows projects to decide which convention plugins they want to use.

Uses a workaround to access jacoco version defined in version catalog:

* declare "..codeSource.location" dependency in buildSrc/build.gradle.kts
* declate `val libs = the<org.gradle.accessors.dm.LibrariesForLibs>()`
# Conflicts:
#	gradle/libs.versions.toml
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.

None yet

1 participant