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

Display whitespace in diff using visible characters #305

Open
honzajavorek opened this issue Dec 8, 2015 · 2 comments
Open

Display whitespace in diff using visible characters #305

honzajavorek opened this issue Dec 8, 2015 · 2 comments
Labels
Epic: CLI reporter Dredd's default reporter

Comments

@honzajavorek
Copy link
Contributor

Coming from a suggestion on API Blueprint Slack channel:

...
As the display for dredd doens't actually show which whitespace is from the parser, and which is formatting from dredd, it's... not easy to catch.
You can wrap it with a hook, which I'll do for the tests, but I'm going to suggest a - whitespace option to display \n, \t, \s or similar for 'eyeball' tests.

So according to the suggestion, dredd's output would use some visible notation for various forms of whitespace so one could easily see the difference in case the only mismatch is in type of whitespace or X spaces vs. tab, etc.

@HowardSterling
Copy link

I'm the person who had the confusion/issue - if anyone needs further clarification.

  • I might suggest \n, \t, and similar 'eyeballing' strings for text output.

@netmilk
Copy link
Contributor

netmilk commented Dec 14, 2015

Totally +1 for this!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Epic: CLI reporter Dredd's default reporter
Projects
None yet
Development

No branches or pull requests

3 participants