-
Notifications
You must be signed in to change notification settings - Fork 103
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 lint rules to prevent CRLF and whitespace errors #2330
Comments
There are some inconsistencies regarding linebreak styles in the repository as shown in #2328 (comment). |
The end_of_line setting of editorconfig might be problematic because it would enforce lf on windows as well. But git can deal with this by itself: with But I have no idea if this is the best approach or if there are some files that should always have crlf (.bat?) while some should always have lf? I wish this was standardised decades ago... |
Also on windows we need to use linefeed for our repo. There should not get any The bat file is an interesting question. I think Call and goto because of labels have a limitation. But I am not sure if the labelparser has still that problem. In the past I had also problems with numbers having crlf without a blank by reading on linux. |
It turned out that editorconfig is sub-optimal for this kind of thing because 1. its Instead, a bit of scripting around git can do all the checking, which I have done in #2334. |
Originally posted by @matrss in #2328 (comment)
With such a setup we could enforce unix-style linebreaks, linebreaks at EOF, and other stuff.
The text was updated successfully, but these errors were encountered: