Skip to content

feat(cli): enhanced summary reporter#6731

Merged
ematipico merged 2 commits intonextfrom
summary-reporter
Jul 9, 2025
Merged

feat(cli): enhanced summary reporter#6731
ematipico merged 2 commits intonextfrom
summary-reporter

Conversation

@ematipico
Copy link
Member

Summary

This PR enhance the summary reporter:

  • added list of files that have lint/assist violations
  • all files have now an hyperlink, so users can open them
  • the of errors/warnings/infos are printed only if they are greater than zero

Test Plan

Updated snapshots, manually tested the links.

@ematipico ematipico requested review from a team July 6, 2025 17:37
@github-actions github-actions bot added A-CLI Area: CLI A-Diagnostic Area: diagnostocis labels Jul 6, 2025
Comment on lines +20 to +22
- index.css
- index.ts
- main.ts
Copy link
Contributor

Choose a reason for hiding this comment

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

How does this work if you're trying out Biome in a new repository that isn't formatted yet? If there are hundreds (or thousands) of files here, it wouldn't be a summary anymore...

Copy link
Member Author

@ematipico ematipico Jul 7, 2025

Choose a reason for hiding this comment

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

Yes, but it's way easier to humanly parse this output, instead of having tons of diagnostics printed, which could eventually exhaust the height of the terminal. It's still a summary of the standard output.

@ematipico ematipico changed the base branch from main to next July 9, 2025 05:52
@ematipico ematipico added this to the Biome v2.2 milestone Jul 9, 2025
@ematipico ematipico merged commit d6a05b5 into next Jul 9, 2025
12 checks passed
@ematipico ematipico deleted the summary-reporter branch July 9, 2025 05:53
ematipico added a commit that referenced this pull request Aug 12, 2025
@github-actions github-actions bot mentioned this pull request Aug 13, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-CLI Area: CLI A-Diagnostic Area: diagnostocis

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants