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

121 add line ending options 1 #132

Closed
wants to merge 2 commits into from
Closed

Conversation

daira
Copy link
Contributor

@daira daira commented Oct 19, 2015

fixes #121

@techtonik
Copy link
Contributor

I am not sure that it is a job of modernize tool to convert linefeeds. This is more like a job for code formatters like go fmt. And even if modernize has this option, I expect it to run and rewrite files even if I didn't run any fixers. Well, actually, these formatting options look like fixers themselves.

@graingert
Copy link
Member

graingert commented Sep 3, 2020

@daira I think this should be done on fissix, also closing in favour of #131

changing lines wholesale can be done with https://github.com/pre-commit/pre-commit-hooks#mixed-line-ending

@graingert graingert closed this Sep 3, 2020
@graingert graingert deleted the 121-add-line-ending-options-1 branch September 3, 2020 23:16
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.

Preserve line separator format (Unix vs. Windows)
3 participants