Skip to content
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

Set MiniCssExtractPlugin ignoreOrder to true for default config #2219

Closed

Conversation

lfloyd117
Copy link

From this discussion in the Ember Discord: https://discord.com/channels/480462759797063690/1308128146058514474/1319687395024830507 with @ef4

The warnings don't particularly mean much when handling CSS via modules, so it should be safe to make the base config ignore them.

@lfloyd117
Copy link
Author

What other steps need to be done in this PR (for versioning or other tasks)? I did not see further references in the contributing doc

Copy link
Collaborator

@NullVoxPopuli NullVoxPopuli left a comment

Choose a reason for hiding this comment

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

Makes sense to me

For others, here are the docs on the option:
https://webpack.js.org/plugins/mini-css-extract-plugin/#remove-order-warnings

@ef4 ef4 changed the base branch from main to stable December 20, 2024 16:29
@ef4 ef4 changed the base branch from stable to main December 20, 2024 16:29
@ef4
Copy link
Contributor

ef4 commented Dec 20, 2024

This should target stable, which means you will want to rebase your commit on stable as well so you're not pulling in all of main.

@NullVoxPopuli
Copy link
Collaborator

NullVoxPopuli commented Dec 20, 2024

@ef4 there is a separate PR targeting stable,
#2220

@NullVoxPopuli
Copy link
Collaborator

Closing in favor of #2220

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.

3 participants