Skip to content

chore: updated the docs to mention correct loader #3353

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

Closed

Conversation

anikethsaha
Copy link
Member

html-loader doesnt uses addDependency method, Might be outdated with the changes
I changed the example with sass-loader and less-loader which does use it

@netlify
Copy link

netlify bot commented Nov 13, 2019

Preview is ready

Built with commit 4c233fa

https://deploy-preview-3353--webpackjsorg-netlify.netlify.com

@anikethsaha
Copy link
Member Author

cc @EugeneHlushko

@montogeek montogeek added the API label Nov 15, 2019
@montogeek
Copy link
Member

Thanks for your contribution, What happened with the yarn.lock? Could you please revert those changes?

@anikethsaha
Copy link
Member Author

that is using yarn install at the beginning.
nothing explicit.

@EugeneHlushko
Copy link
Member

Yes please revert lockfile, can you also mention OS and node version to understand why so many changes in the lockfile after your install

@anikethsaha
Copy link
Member Author

node : v10.15.3
OS : windows 10

@EugeneHlushko
Copy link
Member

Yeap do looks like the changes are related to mac vs win. As soon as its reverted should be good to go

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

Successfully merging this pull request may close these issues.

3 participants