Skip to content

Conversation

@komlev
Copy link
Contributor

@komlev komlev commented Dec 21, 2016

postcss-js parser produce wrapped styles, adding description to fix this problem

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.

Thx again 👍 Just two grammar issues

README.md Outdated
}
```

> If you are using babel 6 compiler and up you need to do the following in order for setup to work
Copy link
Member

@michael-ciniawsky michael-ciniawsky Dec 21, 2016

Choose a reason for hiding this comment

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

  • babel 6 compiler => Babel >= v6
  • in order for setup => in order for the/your setup to work

README.md Outdated

> If you are using babel 6 compiler and up you need to do the following in order for setup to work
> * It is suggested to have only **default** export per style module
Copy link
Member

Choose a reason for hiding this comment

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

only => only one || only => one ?

README.md Outdated
> If you are using babel 6 compiler and up you need to do the following in order for setup to work
> * It is suggested to have only **default** export per style module
> * You need to add [babel-plugin-add-module-exports] to you configuration
Copy link
Member

@michael-ciniawsky michael-ciniawsky Dec 21, 2016

Choose a reason for hiding this comment

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

Maybe switch order here? But I'm nitpicking 🐤 😛

  1. add babel-plugin-add-module-exports
  2. only one default export per module/file

@komlev
Copy link
Contributor Author

komlev commented Dec 21, 2016

Done

@michael-ciniawsky michael-ciniawsky merged commit b66a294 into webpack:master Dec 21, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants