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

CRLF conversion #28

Closed
ForNeVeR opened this issue Aug 29, 2016 · 0 comments · Fixed by #31
Closed

CRLF conversion #28

ForNeVeR opened this issue Aug 29, 2016 · 0 comments · Fixed by #31
Assignees

Comments

@ForNeVeR
Copy link
Member

ForNeVeR commented Aug 29, 2016

From #27:

Unfortunately, I've found that my local repository clone had core.autocrlf set to false (and most of the files were commited to SVN with CRLF anyways, so we'd need to take care of that sooner or later). I've enabled this option now, so for some time it'll be harder for me to review my own pull requests, because there will be many line endings conversions.

We need to convert all the text files ASAP (e.g. from the current point in develop) because it's already starting to create problems in #27.

Convert each top-level directory in a separate commit to make review and testing easier (yes, I'm waiting for tests fails in middle of this).

@ForNeVeR ForNeVeR mentioned this issue Aug 29, 2016
19 tasks
@ForNeVeR ForNeVeR self-assigned this Aug 29, 2016
ForNeVeR added a commit that referenced this issue Aug 30, 2016
ForNeVeR added a commit that referenced this issue Aug 30, 2016
@ForNeVeR ForNeVeR mentioned this issue Aug 30, 2016
2 tasks
ForNeVeR added a commit that referenced this issue Aug 30, 2016
Now they aren't dependent on platform and/or git settings.
ForNeVeR added a commit that referenced this issue Aug 30, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

Successfully merging a pull request may close this issue.

1 participant