solid icon indicating copy to clipboard operation
solid copied to clipboard

dev: add warning when context is undefined

Open Tommypop2 opened this issue 9 months ago • 1 comments

Summary

Using useContext outside of a provider can cause unforeseen and cryptic errors. This PR adds a warning (in dev) when the result of useContext is falsy.

See https://discord.com/channels/722131463138705510/1358899310116213008

Typescript does protect against using an undefined context somewhat, but I thought a warning could be nice, especially for new users.

How did you test this change?

Created a project and consumed a context without being in its provider: image

Warning is printed to the console.

After the project has been built, the warning is not logged.

Tommypop2 avatar Apr 08 '25 13:04 Tommypop2

⚠️ No Changeset found

Latest commit: bc6c19b6d64aa0b45a5a6dbbe1abd7b9bcf0dcf4

Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.

This PR includes no changesets

When changesets are added to this PR, you'll see the packages that this PR includes changesets for and the associated semver types

Click here to learn what changesets are, and how to add one.

Click here if you're a maintainer who wants to add a changeset to this PR

changeset-bot[bot] avatar Apr 08 '25 13:04 changeset-bot[bot]