-
-
Notifications
You must be signed in to change notification settings - Fork 604
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
Update README.md #473
Update README.md #473
Conversation
Codecov Report
@@ Coverage Diff @@
## master #473 +/- ##
======================================
Coverage 98.3% 98.3%
======================================
Files 10 10
Lines 354 354
Branches 79 79
======================================
Hits 348 348
Misses 6 6 Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@briandipalma Thx in advance
README.md
Outdated
@@ -205,8 +205,9 @@ You can configure the generated ident with the `localIdentName` query parameter | |||
} | |||
``` | |||
|
|||
You can also specify the absolute path to your custom `getLocalIdent` function to generate classname based on a different schema. Note that this requires `webpack >= v2.x.` since to be able to pass function in. For example: | |||
You can also specify the absolute path to your custom `getLocalIdent` function to generate classname based on a different schema. This requires `webpack >= v2.x.` (it supports functions in the `options` object). For example: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
webpack >= 2.x
=> webpack >= 2.2.1
bc before that version using functions in options would require to specify an identifier { loader: 'css-loader', options: { ident: 'css-loader', ...options } }
to work properly
Review comment addressed. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@briandipalma Thx
What kind of change does this PR introduce?
Docs
Did you add tests for your changes?
No
If relevant, did you update the README?
Yes
Summary
Previous wording was awkward, not sure the suggested one is perfect so you can reject this.
Does this PR introduce a breaking change?
No
Other information