Skip to content

fix(analyzer): add failed utf conversions to unwanted files#7997

Merged
cx-miguel-dasilva merged 5 commits intomasterfrom
AST-140832
Mar 18, 2026
Merged

fix(analyzer): add failed utf conversions to unwanted files#7997
cx-miguel-dasilva merged 5 commits intomasterfrom
AST-140832

Conversation

@cx-miguel-dasilva
Copy link
Copy Markdown
Collaborator

@cx-miguel-dasilva cx-miguel-dasilva commented Mar 13, 2026

Reason for Proposed Changes

  • files that failed to convert to utf during the analysis process were not being added to unwantad path, and then they would be picked up during getSources of supported parsers, if other platform types were identified.

Proposed Changes

  • add failed conversion to unwanted files
  • add unit test
  • add e2e test

I submit this contribution under the Apache-2.0 license.

@cx-miguel-dasilva cx-miguel-dasilva self-assigned this Mar 13, 2026
@cx-miguel-dasilva cx-miguel-dasilva requested a review from a team as a code owner March 13, 2026 11:33
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 13, 2026

kics-logo

KICS version: v2.1.18

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

cx-artur-ribeiro
cx-artur-ribeiro previously approved these changes Mar 16, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@cx-artur-ribeiro cx-artur-ribeiro left a comment

Choose a reason for hiding this comment

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

LGTM

@cx-miguel-dasilva cx-miguel-dasilva merged commit be60ba3 into master Mar 18, 2026
26 of 27 checks passed
@cx-miguel-dasilva cx-miguel-dasilva deleted the AST-140832 branch March 18, 2026 14:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@cx-artur-ribeiro cx-artur-ribeiro cx-artur-ribeiro approved these changes

Assignees

@cx-miguel-dasilva cx-miguel-dasilva

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@cx-miguel-dasilva @cx-artur-ribeiro