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

Temporary: comment the Surefire configuration #360

Merged
merged 1 commit into from
Feb 18, 2024

Conversation

sleberknight
Copy link
Member

After applying the timezone config to Surefire, suddenly there are no test coverage results. The target/site/jacoco directory simply doesn't exist. This happens locally as well. This is a temporary reversion until I can figure out why adding a Surefire configuration caused there to be no coverage info!

After applying the timezone config to Surefire, suddenly there
are no test coverage results. The target/site/jacoco directory
simply doesn't exist. This happens locally as well. This is
a temporary reversion until I can figure out why adding a
Surefire configuration caused there to be no coverage info!
@sleberknight sleberknight added the investigation Something that needs to be investigated before implementation can proceed label Feb 18, 2024
@sleberknight sleberknight self-assigned this Feb 18, 2024
Copy link

Quality Gate Passed Quality Gate passed

Issues
1 New issue

Measures
0 Security Hotspots
No data about Coverage
0.0% Duplication on New Code

See analysis details on SonarCloud

@sleberknight
Copy link
Member Author

Merging this to bring back the coverage information until I can figure this out.

@sleberknight sleberknight merged commit d5a7145 into master Feb 18, 2024
5 checks passed
@sleberknight sleberknight deleted the temp-comment-out-surefire-config branch February 18, 2024 21:05
@sleberknight sleberknight added this to the 2.1.2 milestone Feb 19, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
investigation Something that needs to be investigated before implementation can proceed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant