Skip to content

fix(dev-middleware): add missing invariant dependency - #49047

Closed
tido64 wants to merge 1 commit into
mainfrom
tido/dev-middleware-missing-dep
Closed

fix(dev-middleware): add missing invariant dependency#49047
tido64 wants to merge 1 commit into
mainfrom
tido/dev-middleware-missing-dep

Conversation

@tido64

Copy link
Copy Markdown
Collaborator

Summary:

dev-middleware uses invariant but does not declare it as a dependency. Under certain hoisting scenarios, or when using pnpm, this will cause dev-middleware to fail while being loaded.

Changelog:

[GENERAL] [FIXED] - add missing invariant dependency

Test Plan:

n/a

@facebook-github-botfacebook-github-bot added CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. p: Microsoft Partner: Microsoft Partner Shared with Meta Applied via automation to indicate that an Issue or Pull Request has been shared with the team. labels Jan 29, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@huntie has imported this pull request. If you are a Meta employee, you can view this diff on Phabricator.

@huntiehuntie left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Thanks!

@facebook-github-botfacebook-github-bot added the Merged This PR has been merged. label Jan 30, 2025
@facebook-github-bot

Copy link
Copy Markdown
Contributor

@huntie merged this pull request in ee8088b.

@react-native-bot

Copy link
Copy Markdown
Collaborator

This pull request was successfully merged by @tido64 in ee8088b

When will my fix make it into a release? | How to file a pick request?

cipolleschi pushed a commit that referenced this pull request Feb 3, 2025
Summary:
`dev-middleware` uses `invariant` but does not declare it as a dependency. Under certain hoisting scenarios, or when using pnpm, this will cause `dev-middleware` to fail while being loaded.
## Changelog:
[GENERAL] [FIXED] - add missing `invariant` dependency
Pull Request resolved: #49047
Test Plan: n/a
Reviewed By: cortinico
Differential Revision: D68835789
Pulled By: huntie
fbshipit-source-id: 13718f4970ed55e6e062b7c2bd719be977abdd0c
cortinico pushed a commit that referenced this pull request Feb 5, 2025
Summary:
`dev-middleware` uses `invariant` but does not declare it as a dependency. Under certain hoisting scenarios, or when using pnpm, this will cause `dev-middleware` to fail while being loaded.
## Changelog:
[GENERAL] [FIXED] - add missing `invariant` dependency
Pull Request resolved: #49047
Test Plan: n/a
Reviewed By: cortinico
Differential Revision: D68835789
Pulled By: huntie
fbshipit-source-id: 13718f4970ed55e6e062b7c2bd719be977abdd0c
@tido64
tido64 deleted the tido/dev-middleware-missing-dep branch February 11, 2025 12:59
robhogan pushed a commit that referenced this pull request Feb 11, 2025
Summary:
`dev-middleware` uses `invariant` but does not declare it as a dependency. Under certain hoisting scenarios, or when using pnpm, this will cause `dev-middleware` to fail while being loaded.
## Changelog:
[GENERAL] [FIXED] - add missing `invariant` dependency
Pull Request resolved: #49047
Test Plan: n/a
Reviewed By: cortinico
Differential Revision: D68835789
Pulled By: huntie
fbshipit-source-id: 13718f4970ed55e6e062b7c2bd719be977abdd0c
linhandev pushed a commit to linhandev/react-native-core that referenced this pull request Jun 26, 2025
Summary:
`dev-middleware` uses `invariant` but does not declare it as a dependency. Under certain hoisting scenarios, or when using pnpm, this will cause `dev-middleware` to fail while being loaded.
## Changelog:
[GENERAL] [FIXED] - add missing `invariant` dependency
Pull Request resolved: react/react-native#49047
Test Plan: n/a
Reviewed By: cortinico
Differential Revision: D68835789
Pulled By: huntie
fbshipit-source-id: 13718f4970ed55e6e062b7c2bd719be977abdd0c
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

CLA SignedThis label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.MergedThis PR has been merged.p: MicrosoftPartner: MicrosoftPartnerShared with MetaApplied via automation to indicate that an Issue or Pull Request has been shared with the team.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@tido64@facebook-github-bot@react-native-bot@huntie