Skip to content

Conversation

@cx-julio-silva
Copy link
Contributor

Proposed Changes

  • The message field in the reports of git scans used to have too much git information that has now been removed in order to make the message clearer. The removed information is still accessible in the artifactLocation field of the report.

Checklist

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

I submit this contribution under the Apache-2.0 license.

@cx-julio-silva cx-julio-silva requested a review from a team as a code owner April 22, 2025 15:21
@github-actions
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 11
Files parsed placeholder 11
Files failed to scan placeholder 0
Total executed queries placeholder 53
Queries failed to execute placeholder 0
Execution time placeholder 1

@github-actions
Copy link

github-actions bot commented May 8, 2025

Logo
Checkmarx One – Scan Summary & Details78bc37bf-dd91-417f-8dbc-2cc7195ff9f7

Policy Management Violations (1)
Policy Name Rule(s) Break Build
FluentAssertions v8 true

@cx-julio-silva cx-julio-silva changed the title changed git report message refactor: changed git report message May 9, 2025
@cx-julio-silva cx-julio-silva merged commit 1ce26d3 into master May 9, 2025
11 checks passed
@cx-julio-silva cx-julio-silva deleted the AST-94168-change-git-report-message branch May 9, 2025 15:58
cx-rogerio-dalot pushed a commit that referenced this pull request Sep 11, 2025
**Proposed Changes**

* The message field in the reports of git scans used to have too much
git information that has now been removed in order to make the message
clearer. The removed information is still accessible in the
artifactLocation field of the report.

**Checklist**

- [ ] I covered my changes with tests.
- [ ] I Updated the documentation that is affected by my changes:
  - [ ] Change in the CLI arguments
  - [ ] Change in the configuration file

I submit this contribution under the Apache-2.0 license.
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.

3 participants