Skip to content

Comments

chore: filter @lwc/engine-server fixture test warnings for lwc:dynamic only#4072

Merged
jmsjtu merged 2 commits intomasterfrom
jtu/enable-engine-server-warnings
Mar 18, 2024
Merged

chore: filter @lwc/engine-server fixture test warnings for lwc:dynamic only#4072
jmsjtu merged 2 commits intomasterfrom
jtu/enable-engine-server-warnings

Conversation

@jmsjtu
Copy link
Member

@jmsjtu jmsjtu commented Mar 17, 2024

Details

In #3381 the @lwc/engine-server fixture test warnings were suppressed.

The intention here was to only supress the warnings generated for lwc:dynamic usage.

Supressing all warnings in @lwc/engine-server hid an issue with the fixture tests in #4055.

Filtering the warnings to specifically look for lwc:dynamic warning messages.

Does this pull request introduce a breaking change?

  • 😮‍💨 No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

  • 🤞 No, it does not introduce an observable change.

GUS work item

@jmsjtu jmsjtu requested a review from a team as a code owner March 17, 2024 22:38
@jmsjtu jmsjtu merged commit 4008342 into master Mar 18, 2024
@jmsjtu jmsjtu deleted the jtu/enable-engine-server-warnings branch March 18, 2024 17:10
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.

4 participants