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

allow use without @react-native-community/cli-server-api #45311

Closed
wants to merge 3 commits into from

Conversation

blakef
Copy link
Contributor

@blakef blakef commented Jul 8, 2024

Summary: Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223

@facebook-github-bot facebook-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: Facebook Partner: Facebook Partner labels Jul 8, 2024
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

@analysis-bot
Copy link

analysis-bot commented Jul 8, 2024

Platform Engine Arch Size (bytes) Diff
android hermes arm64-v8a 21,367,502 +21
android hermes armeabi-v7a n/a --
android hermes x86 n/a --
android hermes x86_64 n/a --
android jsc arm64-v8a 24,564,221 +0
android jsc armeabi-v7a n/a --
android jsc x86 n/a --
android jsc x86_64 n/a --

Base commit: 2eb7bcb
Branch: main

@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

blakef added a commit to blakef/react-native that referenced this pull request Jul 11, 2024
)

Summary:
Pull Request resolved: facebook#45311

Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223
@blakef blakef force-pushed the export-D59377223 branch from 07bf5e2 to 27a953c Compare July 11, 2024 11:42
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

blakef added a commit to blakef/react-native that referenced this pull request Jul 12, 2024
)

Summary:
Pull Request resolved: facebook#45311

Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223
@blakef blakef force-pushed the export-D59377223 branch from 27a953c to 23e216d Compare July 12, 2024 12:28
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

blakef added a commit to blakef/react-native that referenced this pull request Jul 12, 2024
)

Summary:
Pull Request resolved: facebook#45311

Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223
@blakef blakef force-pushed the export-D59377223 branch from 23e216d to 06de1be Compare July 12, 2024 12:34
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

blakef added a commit to blakef/react-native that referenced this pull request Jul 12, 2024
)

Summary:
Pull Request resolved: facebook#45311

Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223
@blakef blakef force-pushed the export-D59377223 branch from 06de1be to 77dfd0a Compare July 12, 2024 12:53
blakef added 3 commits July 15, 2024 03:26
Summary:
Removing the remaining dependencies from the react-native-community/*. This
inlines a copy of the logger.

Changelog:
[Internal][Changed] Removed react-native-community/cli-tools logger dependency

Differential Revision: D59378011
Summary:
Removed the use of version checking and error code that is in react-native-community/cli-tools.

Changelog:
[Internal] [Changed] - Removed community-cli-plugin version & error dependencies

Differential Revision: D59378012
)

Summary:
Pull Request resolved: facebook#45311

Allows Gradle and Xcode builds to continue without the rnc dependency.

Differential Revision: D59377223
@facebook-github-bot
Copy link
Contributor

This pull request was exported from Phabricator. Differential Revision: D59377223

@blakef blakef force-pushed the export-D59377223 branch from 77dfd0a to 6278f2b Compare July 15, 2024 10:32
@facebook-github-bot facebook-github-bot added the Merged This PR has been merged. label Jul 15, 2024
@facebook-github-bot
Copy link
Contributor

This pull request has been merged in 24997dc.

Copy link

This pull request was successfully merged by @blakef in 24997dc

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

blakef added a commit that referenced this pull request Nov 6, 2024
When decoupling the community-cli-plugin from the
@react-native-community/cli-server-api (#45311), a middleware stub was
created to allow a runtime stub to be used in this case. This middleware
should be used so as not to break when the optional cli-server-api
dependency isn't present.

Changelog: [General][Fix] - Fix npm react-native start when
cli-server-api isn't installed

Testing: Forced a runtime exception simulating the package not being
dependent and was able to build rn-tester.
blakef added a commit that referenced this pull request Nov 6, 2024
When decoupling the community-cli-plugin from the
@react-native-community/cli-server-api (#45311), a middleware stub was
created to allow a runtime stub to be used in this case. This middleware
should be used so as not to break when the optional cli-server-api
dependency isn't present.

Changelog: [General][Fix] - Fix npm react-native start when
cli-server-api isn't installed

Testing: Forced a runtime exception simulating the package not being
dependent and was able to build rn-tester.
@blakef blakef mentioned this pull request Nov 6, 2024
facebook-github-bot pushed a commit that referenced this pull request Nov 6, 2024
Summary:
When decoupling the community-cli-plugin from the react-native-community/cli-server-api (#45311), a middleware stub was created to allow a runtime stub to be used in this case. This middleware should be used so as not to break when the optional cli-server-api dependency isn't present.

Changelog:
[General][Fixed] - Fix npm react-native start when cli-server-api isn't installed

Pull Request resolved: #47450

Test Plan:
Forced a runtime exception simulating the package not being dependent and was able to build rn-tester.

![CleanShot 2024-11-06 at 10 49 58@2x](https://github.com/user-attachments/assets/c040ec5b-7000-43bd-ba54-52a672ff3675)

Reviewed By: cipolleschi

Differential Revision: D65532486

Pulled By: blakef

fbshipit-source-id: 2b380607de63ac2da906ef0cb1e48b9ef263cb68
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. fb-exported Merged This PR has been merged. p: Facebook Partner: Facebook Partner
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants