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

Add gitattributes to force eol=lf #1356

Closed
wants to merge 1 commit into from

Conversation

orgads
Copy link
Contributor

@orgads orgads commented Dec 17, 2020

Without this, eslint fails on Windows when core.autocrlf is true

Without this, eslint fails on Windows when core.autocrlf is true
@epoberezkin
Copy link
Member

I think it's better to have environment specific settings in global git config, rather than in projects (there may be some historic noise in .gitignore, but I'd rather we remove it than add more). Also I use windows with git bash - never had this problem...

@orgads
Copy link
Contributor Author

orgads commented Dec 17, 2020

You probably don't have core.autocrlf=true.

@epoberezkin
Copy link
Member

epoberezkin commented Dec 19, 2020

Possibly, either way, let’s not add platform specific configuration - it is better to have in your global git settings

@orgads
Copy link
Contributor Author

orgads commented Dec 19, 2020

Why not?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants