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.

Sign up for GitHub

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

Jump to bottom

Include full PMD report #71

Closed
adangel opened this issue Oct 23, 2020 · 0 comments · Fixed by #92
Closed

Include full PMD report #71

adangel opened this issue Oct 23, 2020 · 0 comments · Fixed by #92
Labels
enhancement New feature or request
Milestone
1.2.0

Comments

@adangel
Copy link
Member

adangel commented Oct 23, 2020 •
edited
Loading

Currently we only show the differences. This assumes, that the baseline is the ground truth. But that might not always be correct.

It would be helpful, if we additionally upload the full report, so one can skim through to verify, that we report correct violations.

Also having a look at any processing error would be beneficial.

Maybe we could even share the logs from the pmd run (warnings, etc.). Currently, the logs are only provided in the builds logs, if PMD execution failed.
@adangel adangel added the enhancement New feature or request label Oct 23, 2020
@adangel adangel mentioned this issue Jun 17, 2021
Merged
@adangel adangel added this to the 1.2.0 milestone Jun 18, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
1.2.0
Development

Successfully merging a pull request may close this issue.

Include full pmd report adangel/pmd-regression-tester
1 participant
@adangel