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

Add file violation count to summary report output #378

Merged
merged 1 commit into from
Sep 19, 2024

Conversation

ermakov-oleg
Copy link
Contributor

What has been done:

  • Added a summary message to display the number of files with violations.
  • If violations are found, the report will now include the total number of violations, the number of files with violations, and the number of files checked.

Example of the summary message:

Found 2 issues in 1 file (checked 1 source file)

Why this is needed:

  • Improves the clarity of the final report by adding information about the number of files with violations.
  • Makes it easier to quickly identify how many files require attention, streamlining the process of addressing issues.

Copy link

netlify bot commented Sep 19, 2024

👷 Deploy request for squawkhq pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit e8ea8e6

Copy link
Owner

@sbdchd sbdchd left a comment

Choose a reason for hiding this comment

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

Nice addition, thank you!

@sbdchd sbdchd added the automerge automerge with kodiak label Sep 19, 2024
@kodiakhq kodiakhq bot merged commit bc010b0 into sbdchd:master Sep 19, 2024
23 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
automerge automerge with kodiak
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants