Skip to content

Conversation

@alan-agius4
Copy link
Collaborator

esbuild will issue a warning when @charset is in the middle of the file. This is caused by css-loader will concats the file and doesn't hoist @charset, (webpack/css-loader#1212).

While, esbuild will issue a warning regarding the above, it will hoist to the very top.

In many cases, this warning is not actionable by the users as the @charset would be likely specified in 3rd party libs.

Closes #22097

…irst rule in the file warning

esbuild will issue a warning when `@charset` is in the middle of the file. This is caused by css-loader will concats the file and doesn't hoist `@charset`, (webpack/css-loader#1212).

While, esbuild will issue a warning regarding the above, it will hoist to the very top.

In many cases, this warning is not actionable by the users as the `@charset` would be likely specified in 3rd party libs.

Closes #22097
@alan-agius4 alan-agius4 added the target: patch This PR is targeted for the next patch release label Nov 5, 2021
@google-cla google-cla bot added the cla: yes label Nov 5, 2021
@alan-agius4 alan-agius4 requested a review from clydin November 5, 2021 08:00
@alan-agius4 alan-agius4 added the action: merge The PR is ready for merge by the caretaker label Nov 8, 2021
@clydin clydin merged commit b3e5888 into angular:master Nov 8, 2021
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators Dec 9, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

warning: "@charset" must be the first rule in the file

2 participants