Skip to content

replaced CRLF with LF line endings - #297

Open
koenpunt wants to merge 1 commit into
jpfuentes2:masterfrom
fetch:fix-line-endings
Open

replaced CRLF with LF line endings#297
koenpunt wants to merge 1 commit into
jpfuentes2:masterfrom
fetch:fix-line-endings

Conversation

@koenpunt

Copy link
Copy Markdown
Collaborator

Replaced line endings to match with rest of the code

@al-the-x

Copy link
Copy Markdown
Collaborator

Ha ha, just commented about this in another issue!

@al-the-x

Copy link
Copy Markdown
Collaborator

As much as I'd love to see this, maybe we should save it until just before 1.2 drops? There may be more code submitted between now and then with the same DOS line-endings that needs to be converted by then... Whenever that is.

@koenpunt

Copy link
Copy Markdown
Collaborator Author

Hm, ok, but these are the only 4 files of 84 left with CRLF. And also git can be configured (on the client) to convert these line endings, so maybe this should be noted in the contribution section of the readme?

@al-the-x

Copy link
Copy Markdown
Collaborator

Good suggestion. Let's save this for just before our 1.2 release and make notes on the contribution page.

@al-the-x al-the-x mentioned this pull request Jul 29, 2013
This was referenced Jul 17, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants