Closed jonathanglasmeyer closed 8 years ago
@jonathanewerner, please only add the first-timers-only
label after the issue has been setup for a first timer (needs an extremely detailed checklist) :-)
Thanks! I hope we can give this to a first-timer!
Ok! :+1: Is there some kind of canonical way to do this?
Edit: tried to come up with a basic checklist. :)
working on it = )
Sent pull request
:octocat: Starter issue
We would like to take the opportunity to invite someone who did not yet contribute to webpack-validator to work on this issue. Follow @YourFirstPR on Twitter for more of these issues across many open source projects :)
:bug: The Problem
We want to support node v0.10, where
path.parse
from the node standard library isn't yet available. So path.parse should be replaced by using the npm module path-parse.Steps:
npm install --save path-parse
tests/passing-configs/index.js
, replacepath.parse
with the function from thepath-parse
package (you'll need to install it by runningnpm install --save-dev path-parse
and then import it at the top of the file).--no-verify
on yourgit commit
and push to ask for help.