Skip to content

Add .gitattributes to .npmignore#1388

Merged
lukeapage merged 1 commit intoless:masterfrom
dpatti:ignore-gitattributes
Jul 20, 2013
Merged

Add .gitattributes to .npmignore#1388
lukeapage merged 1 commit intoless:masterfrom
dpatti:ignore-gitattributes

Conversation

@dpatti
Copy link
Copy Markdown
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
Copy Markdown
Contributor

Isn't this an issue with hawk?

@dpatti
Copy link
Copy Markdown
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
Copy Markdown
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