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

Kics incorrectly reporting the number of errors in summary #3392

Closed
TommyE123 opened this issue Feb 25, 2024 · 2 comments
Closed

Kics incorrectly reporting the number of errors in summary #3392

TommyE123 opened this issue Feb 25, 2024 · 2 comments
Labels
bug Something isn't working

Comments

@TommyE123
Copy link
Contributor

Describe the bug
The Kics linter is incorrectly reporting the number of errors. This discrepancy becomes apparent when running the linter on files with multiple warnings or errors, as MegaLinter incorrectly displays "1" error instead of the actual count in the summary.

To Reproduce
Run the Kics linter on a file containing several warnings or errors.
Observe the output from MegaLinter, which inaccurately reports "1" error.

Expected behavior
MegaLinter should accurately count and report the total number of errors within the file.

@TommyE123 TommyE123 added the bug Something isn't working label Feb 25, 2024
@nvuillam
Copy link
Member

nvuillam commented Feb 28, 2024

@TommyE123 indeed !

A linter descriptor must contain properties cli_lint_errors_count and cli_lint_errors_regex to be able to count the errors by getting info from the linter output log.

Maybe you'd like to submit a PR to add them on KICS descriptor ? :)

Example:

    cli_lint_errors_count: regex_number
    cli_lint_errors_regex: "files: ([0-9]+) finding"

@TommyE123
Copy link
Contributor Author

The issue has been resolved and is now working as expected in version 7.10.0 following the latest release.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants