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

Improve output of TextReporter #680

Open
wants to merge 40 commits into
base: main
Choose a base branch
from
Open

Conversation

piranna
Copy link
Contributor

@piranna piranna commented May 4, 2022

Three years ago I did some improvements of TextReporter allowing it to adjust better to terminals width and show more missing lines. Here I've done multiple improvements based on personal experience during this time:

  • Trim whitespaces in missing lines column. This allow to don't affect report output if terminal gets resized, just only for the long lines
  • split missing lines with whitespaces instead of commas. This improve legibility while not affecting number of visible missing lines
  • show continuation dots for full ranges, not truncating any of them in half. This removes some info from missing lines column, but prevent confusions about the actual missing lines numbers.

Earned 3 extra chars for `MISSING_COL`
Use 80 columns width by default, unlimited if `maxCols` option is set
explicitly to zero
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.

None yet

1 participant