Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Prevent .gitignore from ignoring debug module #1908

Closed

Conversation

zkat
Copy link
Contributor

@zkat zkat commented Jun 5, 2015

On case insensitive platforms, the rule was catching the node module
under npm and eslint.

This should fix the issue, and make #1899 mergeable.

cc: @othiym23 @Fishrock123

On case insensitive platforms, the rule was catching the node module
under npm and eslint.

This should fix the issue, and make nodejs#1899 mergeable.
Fishrock123 pushed a commit that referenced this pull request Jun 5, 2015
On case insensitive platforms, the rule was catching the debug module
under npm and eslint.

See: #1899 (comment)
PR-URL: #1908
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
@Fishrock123
Copy link
Contributor

Thanks, landed in 02c3450 :)

@Fishrock123 Fishrock123 closed this Jun 5, 2015
@rvagg rvagg mentioned this pull request Jun 11, 2015
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants