-
Notifications
You must be signed in to change notification settings - Fork 59
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 handling of trailing whitespace in API dump files #133
Labels
Comments
What about just removing the last whitespace? |
I'd appreciate this too, I also have my IntelliJ set up this way. For me this mostly comes up during merges. Removing the last |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Currently, any API dump that BCV produces contains two trailing whitespaces lines,
\n\n
.I have enabled 'strip trailing whitespace' in IntelliJ https://stackoverflow.com/questions/20679842/remove-trailing-whitespace-on-save-in-intellij-idea-12
When I open a .api file, which might automatically happen during a commit dialog, IntelliJ will replace the two trailing
\n\n
with a single\n
. This is annoying, because then BCV will fail the test.Other editors might also have problems with different types of behaviour that are difficult to predict or control.
Could BCV be changed to ignore any difference in trailing whitespace?
The text was updated successfully, but these errors were encountered: