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

AST-36384 add support for improved-scan-report invocation #688

Merged
merged 5 commits into from
Apr 5, 2024

Conversation

cx-joses
Copy link
Contributor

@cx-joses cx-joses commented Mar 27, 2024

By submitting a PR to this repository, you agree to the terms within the Checkmarx Code of Conduct. Please see the contributing guidelines for how to create and submit a high-quality PR for this repo.

Description

Added support to call for new improved-scan-report pdf report instead of scan-report in case of enabled NEW_SAST_SCAN_REPORT_ENABLED feature flag

References

https://checkmarx.atlassian.net/browse/AST-36384

Testing

Manual and unit tests

Checklist

  • All active GitHub checks for tests, formatting, and security are passing
  • The correct base branch is being used

@pedrompflopes pedrompflopes requested review from a team and margaritalm and removed request for a team March 27, 2024 14:54
Copy link

github-actions bot commented Mar 27, 2024

Logo
Checkmarx One – Scan Summary & Detailsd9c23dd8-bf53-4de0-9467-03c61021cbe1

Policy Management Violations

Policy Name Rule(s) Break Build
[SAST-ML0] Not allowed NEW Sast vulnerabilities true

No New Or Fixed Issues Found

@tiagobcx tiagobcx requested a review from OrShamirCM April 2, 2024 14:20
@cx-joses cx-joses force-pushed the feature/AST-36384-support-improved-report branch 2 times, most recently from 3b426cf to bff9501 Compare April 4, 2024 17:23
@cx-joses cx-joses force-pushed the feature/AST-36384-support-improved-report branch from bff9501 to be709b3 Compare April 5, 2024 10:50
@cx-joses cx-joses merged commit 5b44ada into main Apr 5, 2024
8 checks passed
@cx-joses cx-joses deleted the feature/AST-36384-support-improved-report branch April 5, 2024 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants