Skip to content

Conversation

@federico-vst
Copy link

Pushing two small fixes to the PDF report.
In particular, the Risk Rules checked section is incomplete, while it is correctly populated in the adoc report.
This is because the riskRules map was not passed to the pdfReporter constructor.

@ezavgorodniy
Copy link
Collaborator

lgtm

@ezavgorodniy
Copy link
Collaborator

smth with GoSec check

@ezavgorodniy ezavgorodniy self-assigned this Nov 19, 2025
@ezavgorodniy ezavgorodniy self-requested a review November 19, 2025 11:17
@federico-vst
Copy link
Author

smth with GoSec check

Yes, an underlying issue in colors.go, not affected by this PR. It was recently fixed in this commit: 293892c with a new version of colors.go, now already in master.
I don't have that commit in my fork, I have the old version of colors.go, but it should be fixed when merging

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