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 .npmignore #1388

Merged
merged 1 commit into from
Jul 20, 2013
Merged

Add .gitattributes to .npmignore #1388

merged 1 commit into from
Jul 20, 2013

Conversation

dpatti
Copy link
Contributor

@dpatti dpatti commented Jun 27, 2013

One of the dependencies, hawk, uses CRLF line endings. If you use less
in your project and check in node_modules to source control, git sees
the .gitattributes that is packaged with less and obeys it for that
subdirectory. This gives you line ending conflicts with hawk.

.gitattributes is not relevant to the packaged module and should not be
included anyway.

One of the dependencies, hawk, uses CRLF line endings. If you use less
in your project and check in node_modules to source control, git sees
the .gitattributes that is packaged with less and obeys it for that
subdirectory. This gives you line ending conflicts with hawk.

.gitattributes is not relevant to the packaged module and should not be
included anyway.
@evocateur
Copy link
Contributor

Isn't this an issue with hawk?

@dpatti
Copy link
Contributor Author

dpatti commented Jul 6, 2013

I don't know if I could consider using CRLF line endings an issue. The problem here is that by installing less through npm, you are getting a repository-specific file that is influencing the parent project's repository, assuming they use git.

@lukeapage
Copy link
Member

Yeh, I plan to get this fixed.

lukeapage added a commit that referenced this pull request Jul 20, 2013
@lukeapage lukeapage merged commit 67bd5a2 into less:master Jul 20, 2013
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants