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

[Android] Prepare auto-linking for double publishing #2076

Conversation

cipolleschi
Copy link
Contributor

@cipolleschi cipolleschi commented Sep 14, 2023

Summary:

With this change, we are preparing the CLI to support double publishing of React Native.
This change should be transparent to our users until we set up a version of React Native that is different from 0.X.Y

Test Plan:

Tested the code with RNTester.

Checklist

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

@cipolleschi cipolleschi force-pushed the cipolleschi/support_new_arch_enabled_from_version branch from 892eefc to b4c4242 Compare September 14, 2023 09:12
@cipolleschi cipolleschi marked this pull request as ready for review September 14, 2023 09:35
@cipolleschi cipolleschi force-pushed the cipolleschi/support_new_arch_enabled_from_version branch from b4c4242 to 9462af6 Compare September 14, 2023 10:15
Copy link
Member

@cortinico cortinico left a comment

Choose a reason for hiding this comment

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

LGTM 👍

@thymikee thymikee merged commit d695144 into react-native-community:main Sep 14, 2023
6 checks passed
@cipolleschi cipolleschi deleted the cipolleschi/support_new_arch_enabled_from_version branch September 14, 2023 12:36
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Sep 18, 2023
Summary:
This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: f22b912f67bc67d4b0b544371b1c334a92892666
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Sep 18, 2023
Summary:
Pull Request resolved: facebook#39502

This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: 55f324ac4dec0e23af8b231bb85703d9e53d95e0
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Sep 18, 2023
Summary:
Pull Request resolved: facebook#39502

This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: 010c735b77c6bcbff5b442d1a454571209738c5d
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Sep 18, 2023
Summary:
Pull Request resolved: facebook#39502

This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: 0a829d3aa75ecd7fb45c1f40b03f5a959744f92e
cipolleschi added a commit to cipolleschi/react-native that referenced this pull request Sep 18, 2023
Summary:
Pull Request resolved: facebook#39502

This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: 1dca14650a2a4de5b7d7caaa4043bb80845436d5
facebook-github-bot pushed a commit to facebook/react-native that referenced this pull request Sep 18, 2023
Summary:
Pull Request resolved: #39502

This diff adds the parsing of the package.json to retrieve the version of react native and it matches it against a regex to decide whether to enable or not the New Architecture.

This change depends on [this CLI's PR](react-native-community/cli#2076).

## Changelog:
[Android][Added] - Parse RN Version to decide whether to enable the New Arch or not.

Reviewed By: cortinico

Differential Revision: D49270012

fbshipit-source-id: 5164a1ddd26f1822c0d717a7a9e2416ec38d78a2
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.

3 participants