-
Notifications
You must be signed in to change notification settings - Fork 24
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
chore(deps): update dependency css-loader to v7 #4612
Conversation
🦋 Changeset detectedLatest commit: 0c84b0b The changes in this PR will be included in the next version bump. This PR includes changesets to release 0 packagesWhen changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types Not sure what this means? Click here to learn what changesets are. Click here if you're a maintainer who wants to add another changeset to this PR |
8931eb7
to
ad57528
Compare
5bbd3c7
to
889a7f5
Compare
Edited/Blocked NotificationRenovate will not automatically rebase this PR, because it does not recognize the last commit author and assumes somebody else may have edited the PR. You can manually request rebase by checking the rebase/retry box above. |
5e812da
to
cef5265
Compare
cef5265
to
9ab978b
Compare
I've discovered that the the underlying issue cause some of the CSS to not be present int he DOM is due to us having a mix of class name conventions in our code. With the update As we can see, there variants that have kebab classes like The solution to this is adding the flag |
Calling out a good point @HeartSquared raised, it would be better to have class names that follow a single convention, ie: camel case, but maybe we release this as a fix first in case there are implication of folks who may be targeting our classes in things like classNameOverride |
9ab978b
to
276dba4
Compare
This PR contains the following updates:
^6.11.0
->^7.1.1
Release Notes
webpack-contrib/css-loader (css-loader)
v7.1.1
Compare Source
v7.1.0
Compare Source
Features
getJSON
option to output CSS modules mapping (#1577) (af834b4)v7.0.0
Compare Source
⚠ BREAKING CHANGES
modules.namedExport
option istrue
by default if you enable theesModule
optionMigration guide:
Before:
After:
To restore 6.x behavior, please use:
modules.exportLocalsConvention
has the valueas-is
when themodules.namedExport
option istrue
and you don't specify a value5.27.0
18.12.0
Features
modules.namedExports
option works fine with anymodules.exportLocalsConvention
values (f96a110)modules.exportLocalsConvention
options (40e1668)Configuration
📅 Schedule: Branch creation - At any time (no schedule defined), Automerge - At any time (no schedule defined).
🚦 Automerge: Disabled by config. Please merge this manually once you are satisfied.
♻ Rebasing: Whenever PR becomes conflicted, or you tick the rebase/retry checkbox.
👻 Immortal: This PR will be recreated if closed unmerged. Get config help if that's undesired.
This PR has been generated by Renovate Bot.