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

feat(report): pdf report #3488 #3556

Merged
merged 12 commits into from
Jun 9, 2021
Merged

feat(report): pdf report #3488 #3556

merged 12 commits into from
Jun 9, 2021

Conversation

rogeriopeixotocx
Copy link
Contributor

@rogeriopeixotocx rogeriopeixotocx commented Jun 4, 2021

closes #3488

Proposed Changes

  • PDF report

I submit this contribution under the Apache-2.0 license.

@rogeriopeixotocx rogeriopeixotocx added the go Pull requests that update Go code label Jun 4, 2021
@rogeriopeixotocx rogeriopeixotocx added this to the Reporting milestone Jun 4, 2021
@rogeriopeixotocx rogeriopeixotocx requested a review from a team June 4, 2021 18:29
@rogeriopeixotocx rogeriopeixotocx self-assigned this Jun 4, 2021
@kicsbot
Copy link
Contributor

kicsbot commented Jun 4, 2021

Scan submitted to Checkmarx

@kicsbot
Copy link
Contributor

kicsbot commented Jun 4, 2021

Logo
Checkmarx SAST - Scan Summary & Details

Cx-SAST Summary

Total of 0 vulnerabilities
High 0 High
Medium 0 Medium
Low 0 Low
Info 0 Info

Violation Summary

No policy violation found

@kicsbot
Copy link
Contributor

kicsbot commented Jun 8, 2021

Scan not submitted to Checkmarx due to existing Active scan for the same project.

@rogeriopeixotocx rogeriopeixotocx marked this pull request as ready for review June 8, 2021 17:13
@rogeriopeixotocx rogeriopeixotocx changed the title feat(report): pdf report POC feat(report): pdf report Jun 8, 2021
@rogeriopeixotocx rogeriopeixotocx force-pushed the feature/pdf-report branch 3 times, most recently from 86718ca to 04cf615 Compare June 9, 2021 08:19
Signed-off-by: Rogério Peixoto <[email protected]>
@rogeriopeixotocx rogeriopeixotocx changed the title feat(report): pdf report feat(report): pdf report #3488 Jun 9, 2021
Copy link
Contributor

@felipe-avelar felipe-avelar left a comment

Choose a reason for hiding this comment

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

Please, check comments

pkg/report/pdf.go Show resolved Hide resolved
pkg/report/pdf.go Show resolved Hide resolved
pkg/report/pdf.go Show resolved Hide resolved
pkg/report/pdf.go Outdated Show resolved Hide resolved
pkg/report/pdf.go Show resolved Hide resolved
pkg/report/pdf.go Outdated Show resolved Hide resolved
Signed-off-by: Rogério Peixoto <[email protected]>
felipe-avelar
felipe-avelar previously approved these changes Jun 9, 2021
Copy link
Contributor

@felipe-avelar felipe-avelar left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Collaborator

@joaoReigota1 joaoReigota1 left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@felipe-avelar felipe-avelar left a comment

Choose a reason for hiding this comment

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

LGTM

@rogeriopeixotocx rogeriopeixotocx merged commit 44e30bd into master Jun 9, 2021
@rogeriopeixotocx rogeriopeixotocx deleted the feature/pdf-report branch June 9, 2021 14:58
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
go Pull requests that update Go code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Reporting - add PDF report format
4 participants