Skip to content

Conversation

@marklundin
Copy link
Collaborator

There are a number of deprecated properties in the engine which issues warnings. This PR simply supresses those warnings during validation.

@marklundin marklundin requested a review from a team September 23, 2025 13:16
@marklundin marklundin self-assigned this Sep 23, 2025
@changeset-bot
Copy link

changeset-bot bot commented Sep 23, 2025

🦋 Changeset detected

Latest commit: e7270a6

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 2 packages
Name Type
@playcanvas/react Patch
@playcanvas/blocks Patch

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

@pkg-pr-new
Copy link

pkg-pr-new bot commented Sep 23, 2025

Open in StackBlitz

npm i https://pkg.pr.new/@playcanvas/blocks@257
npm i https://pkg.pr.new/@playcanvas/react@257

commit: e7270a6

Copy link

@mvaligursky mvaligursky left a comment

Choose a reason for hiding this comment

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

Seems ok.
This will not hide other useful warnings I assume, as it's used only in the validation time?

Is this still needed? playcanvas/engine#7964

@marklundin
Copy link
Collaborator Author

Yep it only happens at startup an only overriding the warn, so there's a chance it might miss some warnings. Something like #7964 would be more robust, but this will be ok for now

@marklundin marklundin merged commit 98dc81c into main Sep 23, 2025
7 checks passed
@marklundin marklundin deleted the fix-suppress-dep-warnings branch September 23, 2025 16:01
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants