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

Exclude files specified in .jshintignore#17

Open
ppissanetzky wants to merge 1 commit into
webpack-contrib:masterfrom
ppissanetzky:master
Open

Exclude files specified in .jshintignore#17
ppissanetzky wants to merge 1 commit into
webpack-contrib:masterfrom
ppissanetzky:master

Conversation

@ppissanetzky
Copy link
Copy Markdown

No description provided.

@jvmccarthy
Copy link
Copy Markdown

+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
Copy Markdown
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
Copy Markdown

😅 @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
Copy Markdown
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.

4 participants