Skip to content
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

breaking: Remove secondary exports of cli-server-api #2584

Merged
merged 1 commit into from
Jan 27, 2025

Conversation

huntie
Copy link
Collaborator

@huntie huntie commented Jan 27, 2025

Summary:

Around #2583, I'm taking the opportunity to review the API boundary of the cli-server-api package.

  • This PR is an opinionated narrowing of the API exported by this package.
  • Functionally, indexPageMiddleware is moved into the middleware stack returned by createDevServerMiddleware.

Test Plan:

✅ Builds
✅ GitHub-wide code search to look for references to these imports outside of react-native.

Checklist

  • Documentation is up to date to reflect these changes.
  • Follows commit message convention described in CONTRIBUTING.md

@thymikee thymikee merged commit 5c2764f into react-native-community:main Jan 27, 2025
4 of 8 checks passed
@huntie huntie deleted the cli-server-api-exports branch January 28, 2025 10:19
huntie added a commit to huntie/react-native that referenced this pull request Jan 28, 2025
Summary:
Follows react-native-community/cli#2584.

- Also add FIXME comment flagging potential core APIs gap without CLI.

Changelog: [Internal]

Differential Revision: D68766565
huntie added a commit to huntie/react-native that referenced this pull request Jan 28, 2025
Summary:

Follows react-native-community/cli#2584.

- Also add FIXME comment flagging potential core APIs gap without CLI.

Changelog: [Internal]

Differential Revision: D68766565
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Jan 28, 2025
Summary:
Pull Request resolved: #48997

Follows react-native-community/cli#2584.

- Also add FIXME comment flagging potential core APIs gap without CLI.

Changelog: [Internal]

Reviewed By: cortinico

Differential Revision: D68766565

fbshipit-source-id: 60747715f76c4323e306c39ab0613fb4818b4914
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.

2 participants