Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Suggested change to readme #535

Merged
merged 1 commit into from
Aug 30, 2016
Merged

Suggested change to readme #535

merged 1 commit into from
Aug 30, 2016

Conversation

dannyc5
Copy link
Contributor

@dannyc5 dannyc5 commented Aug 29, 2016

Took a while to hunt down why my config wasn't working and figured out that I needed to add this dependency only after reading through #238 and noticing this line in a linked example.

I think adding this line may add some clarity and save time for several other webpack / eslint noobs (like me 😄 ).

Took a while to hunt down why my config wasn't working and figured out that I needed to add this dependency only after reading through [import-js#238](import-js#238) and noticing [this line in a linked example](https://gist.github.com/ravasthi/abcfee465411fc45a8bc28decb9d8e5e#file-package-json-L24). I think adding this line may add some clarity and save time for several other webpack / eslint noobs (like me).
@coveralls
Copy link

coveralls commented Aug 29, 2016

Coverage Status

Coverage remained the same at 97.729% when pulling af40e33 on danielchangNYC:patch-1 into 8da6a5a on benmosher:master.

@benmosher
Copy link
Member

Yeah, this has come up a few times in different issues. I like the way you've addressed it, thanks! 😄

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

Successfully merging this pull request may close these issues.

3 participants