Skip to content

Initial Code Coverage Configuration#15668

Closed
jdaugherty wants to merge 2 commits into
8.0.x-hibernate7from
feature/codecoverage
Closed

Initial Code Coverage Configuration#15668
jdaugherty wants to merge 2 commits into
8.0.x-hibernate7from
feature/codecoverage

Conversation

@jdaugherty

Copy link
Copy Markdown
Contributor

Initial Code Coverage Configuration - requires 8.0.x-hibernate7 merge first

@jdaugherty

Copy link
Copy Markdown
Contributor Author

This is a test of the code coverage job - I'm still not sure if we should just have all of our existing CI tests run coverage or if it should be separate.

@codecov

codecov Bot commented May 18, 2026

Copy link
Copy Markdown

Welcome to Codecov 🎉

Once you merge this PR into your default branch, you're all set! Codecov will compare coverage reports and display results in all future pull requests.

Thanks for integrating Codecov - We've got you covered ☂️

@testlens-app

testlens-app Bot commented May 18, 2026

Copy link
Copy Markdown

✅ All tests passed ✅

🏷️ Commit: 7ace696
▶️ Tests: 29421 executed
⚪️ Checks: 43/43 completed


Learn more about TestLens at testlens.app.

borinquenkid pushed a commit that referenced this pull request Jun 1, 2026
Adds a grails-jacoco convention plugin that configures JaCoCo per subproject
and registers a root jacocoAggregateReport task producing a single XML report
for Codecov, handling the hibernate5/hibernate7 duplicate-class case. Applies
grails-jacoco to 96 subprojects and adds the Coverage workflow that uploads to
the existing codecov.yml configuration.

Layers on top of PR-G (grails-code-analysis / grails-violation-aggregation),
which provides the build-logic test infrastructure and plugin registration
scaffold. Folds in the standalone coverage work from #15668.

Extracted from the Hibernate 7 staging branch (PR #15654) per review feedback
that JaCoCo should be introduced as its own focused PR for the project to
review independently.

Assisted-by: claude-code:claude-4.7-opus
@jdaugherty

Copy link
Copy Markdown
Contributor Author

Closing since @jamesfredley pulled forward these changes in #15687

@jdaugherty jdaugherty closed this Jun 2, 2026
jamesfredley pushed a commit to codeconsole/grails-core that referenced this pull request Jun 3, 2026
Adds a grails-jacoco convention plugin that configures JaCoCo per subproject
and registers a root jacocoAggregateReport task producing a single XML report
for Codecov, handling the hibernate5/hibernate7 duplicate-class case. Applies
grails-jacoco to 96 subprojects and adds the Coverage workflow that uploads to
the existing codecov.yml configuration.

Layers on top of PR-G (grails-code-analysis / grails-violation-aggregation),
which provides the build-logic test infrastructure and plugin registration
scaffold. Folds in the standalone coverage work from apache#15668.

Extracted from the Hibernate 7 staging branch (PR apache#15654) per review feedback
that JaCoCo should be introduced as its own focused PR for the project to
review independently.

Assisted-by: claude-code:claude-4.7-opus
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.

1 participant