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

Collect coverage reports with builds on Jenkins #623

Merged
merged 5 commits into from
Apr 15, 2022

Conversation

pzygielo
Copy link
Contributor

@pzygielo pzygielo commented Apr 15, 2022

graph

@pzygielo pzygielo closed this Apr 15, 2022
@pzygielo pzygielo deleted the jacoco branch April 15, 2022 12:57
@pzygielo pzygielo restored the jacoco branch April 15, 2022 14:14
@pzygielo pzygielo reopened this Apr 15, 2022
Bumps [jacoco-maven-plugin](https://github.com/jacoco/jacoco) from 0.7.4.201502262128 to 0.8.8.
- [Release notes](https://github.com/jacoco/jacoco/releases)
- [Commits](https://github.com/jacoco/jacoco/commits/v0.8.8)

---
updated-dependencies:
- dependency-name: org.jacoco:jacoco-maven-plugin
  dependency-type: direct:production
  update-type: version-update:semver-minor
...

Signed-off-by: dependabot[bot] <[email protected]>
@pzygielo pzygielo marked this pull request as ready for review April 15, 2022 14:35
@pzygielo
Copy link
Contributor Author

May I ask for review, please?

}
}
sh 'mvn clean install -P jacoco'
junit testResults: '**/target/surefire-reports/*.xml', allowEmptyResults: true
Copy link
Contributor

Choose a reason for hiding this comment

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

Not sure if it is used in hk2, probably not, I usually use *-reports, as failsafe plugin uses it's own directory. If failsafe is not used here, just ignore my comment ;)

@dmatej dmatej merged commit 0e66aa3 into eclipse-ee4j:master Apr 15, 2022
@pzygielo pzygielo deleted the jacoco branch April 15, 2022 17:01
@dmatej dmatej added this to the 3.0.4 milestone Mar 27, 2023
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