Skip to content

[mcp] Fix package.json #33101

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 1 commit into from
May 2, 2025
Merged

[mcp] Fix package.json #33101

merged 1 commit into from
May 2, 2025

Conversation

poteto
Copy link
Member

@poteto poteto commented May 2, 2025

Since we use esbuild we need to correctly move dependencies that are required at runtime into dependencies and other packages that are only used in development in to devDependencies. This ensures the correct packages are included in the build.

Stack created with Sapling. Best reviewed with ReviewStack.

Since we use esbuild we need to correctly move dependencies that are required at runtime into `dependencies` and other packages that are only used in development in to `devDependencies`. This ensures the correct packages are included in the build.
@github-actions github-actions bot added the React Core Team Opened by a member of the React Core Team label May 2, 2025
@poteto poteto marked this pull request as ready for review May 2, 2025 17:37
@poteto poteto merged commit e5f0315 into main May 2, 2025
29 checks passed
@poteto poteto deleted the pr33101 branch May 2, 2025 17:56
poteto added a commit that referenced this pull request May 2, 2025
Adds typescript support.
---
[//]: # (BEGIN SAPLING FOOTER)
Stack created with [Sapling](https://sapling-scm.com). Best reviewed
with [ReviewStack](https://reviewstack.dev/facebook/react/pull/33082).
* #33085
* #33084
* #33083
* __->__ #33082
* #33101
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed React Core Team Opened by a member of the React Core Team
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants