Skip to content
This repository has been archived by the owner on Jul 18, 2019. It is now read-only.

Exclude files specified in .jshintignore #17

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

ppissanetzky
Copy link

No description provided.

@jvmccarthy
Copy link

+1 Just ran into this too. Please consider merging this PR. It would be nice to have this jshint feature supported in the jshint-loader. Thanks @sokra for your work on webpack and these loaders and @ppissanetzky for this PR!

@sokra
Copy link
Member

sokra commented Nov 18, 2015

@jvmccarthy Do you want to maintain this loader? I'm unable to decide this from code and don't have the time to test it.

@jvmccarthy
Copy link

😅 @sokra I really appreciate all your efforts on webpack and would love to help. I use this loader in my builds and can help maintain it. I don't know much about loader development, but I can come up to speed on it.

I'd like to put some tests in place for this loader if I'm going to maintain it. Do you have an example of a loader that has a test suite?

Thanks again,
-John

@michael-ciniawsky
Copy link
Member

@ppissanetzky Please close and reopen the PR to trigger the CLA Bot again and rebase against current master

@jvmccarthy Still interested in maintaining this loader, would be appreciated, we can set you up 😛 ?

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

Successfully merging this pull request may close these issues.

None yet

4 participants