Skip to content
This repository has been archived by the owner on Aug 20, 2018. It is now read-only.

docs(README): add warning about webpack 2 usage #49

Merged
merged 2 commits into from
May 7, 2017

Conversation

mkarajohn
Copy link
Contributor

Currently the note, that this loader's functionality is on by default in Webpack 2, is not visible in the documentation page.

Moved it under its own section, hoping it'll now be visible.

Currently the note, that this loader's functionality is on by default in Webpack 2, is not visible in the [documentation page](https://webpack.js.org/loaders/json-loader/#components/sidebar/sidebar.jsx).

Moved it under its own section, hoping it'll now be visible
Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@mkarajohn See comment 😛

README.md Outdated
**Note: Since webpack v2, JSON files will work by default. You might still want to use this if you use a custom file extension.**
<h2 align="center">Note</h2>

**Since webpack v2, JSON files will work by default. You might still want to use this if you use a custom file extension.**
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

You could add ⚠️ as prefix || && appendix an move it directly under the Install section, but please don't add another top-level section, since we try to achieve docs consistency accross webpack-contrib

Also tweaked the text a little, for clarity
@mkarajohn
Copy link
Contributor Author

@michael-ciniawsky Updated, let me know if it is ok now!

Copy link
Member

@michael-ciniawsky michael-ciniawsky left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@michael-ciniawsky michael-ciniawsky changed the title Update README.md docs(README): add warning about webpack 2 usage May 7, 2017
@michael-ciniawsky michael-ciniawsky merged commit 62e9f34 into webpack-contrib:master May 7, 2017
sokra added a commit that referenced this pull request May 7, 2017
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.

3 participants