Skip to content

Conversation

@andrzej-janczak
Copy link
Contributor

@andrzej-janczak andrzej-janczak commented Apr 8, 2025

When running like:
cli-v2 analyze --format sarif --output results.sarif then all tools declared in codacy yaml will run and output will be merged into one sarif file

- Moved the `mergeSarifOutputs` function from `analyze.go` to `utils/sarif.go` for better code organization.
- Updated the `analyze.go` file to use the new utility function for merging SARIF outputs.
- Enhanced the `Rule` struct in the utils package to use `map[string]interface{}` for properties.
@codacy-production
Copy link

codacy-production bot commented Apr 8, 2025

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-0.65% 0.00%
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (93219d2) 2260 600 26.55%
Head commit (3cdee94) 2317 (+57) 600 (+0) 25.90% (-0.65%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#62) 76 0 0.00%

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

@alerizzo alerizzo deleted the PLUTO-1385-analyze-run-all branch June 3, 2025 09:44
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