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: ast 40209 added validation status to sarif report #238

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

diogo-fjrocha
Copy link
Collaborator

@diogo-fjrocha diogo-fjrocha commented Apr 19, 2024

Proposed Changes

Added the property bag to Sarif Results ("properties"), in accordance with SARIF guidelines.
Added validationStatus to properties of each result in SARIF, in line with what already happens in JSON.

Checklist

  • I covered my changes with tests (not applicable).
  • I Updated the documentation that is affected by my changes (not applicable):
    • Change in the CLI arguments
    • Change in the configuration file

I submit this contribution under the Apache-2.0 license.

Copy link

kics-logo

KICS version: v1.7.13

Category Results
HIGH HIGH 0
MEDIUM MEDIUM 0
LOW LOW 0
INFO INFO 0
TRACE TRACE 0
TOTAL TOTAL 0
Metric Values
Files scanned placeholder 8
Files parsed placeholder 8
Files failed to scan placeholder 0
Total executed queries placeholder 53
Queries failed to execute placeholder 0
Execution time placeholder 1

@diogo-fjrocha diogo-fjrocha changed the title AST-40209-Added validationStatus to SARIF report feat: AST-40209-Added validationStatus to SARIF report Apr 19, 2024
@diogo-fjrocha diogo-fjrocha changed the title feat: AST-40209-Added validationStatus to SARIF report feat: Ast 40209 added validation status to sarif report Apr 19, 2024
@diogo-fjrocha diogo-fjrocha changed the title feat: Ast 40209 added validation status to sarif report feat: added validation status to sarif report Apr 19, 2024
@diogo-fjrocha diogo-fjrocha changed the title feat: added validation status to sarif report feat: ast 40209 added validation status to sarif report Apr 19, 2024
@diogo-fjrocha diogo-fjrocha added this pull request to the merge queue Apr 22, 2024
Merged via the queue into master with commit e2925c9 Apr 22, 2024
10 of 14 checks passed
@diogo-fjrocha diogo-fjrocha deleted the AST-40209-add-validation-sarif branch April 22, 2024 14:51
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