Skip to content

Conversation

szymonrybczak
Copy link
Collaborator

@szymonrybczak szymonrybczak commented Nov 8, 2023

Summary:

This option was deprecated in v3, right now we have v12. More context https://github.com/react-native-community/cli/pull/768/files#r333437015 :)

Test Plan:

  1. Clone the repository and do all the required steps from the Contributing guide
  2. Run this command:
node /path/to/react-native-cli/packages/cli/build/bin.js run-android

Build should succeed.

Checklist

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

Copy link
Member

@thymikee thymikee left a comment

Choose a reason for hiding this comment

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

LGMT, @cortinico please check if that's not breaking for anybody

@szymonrybczak
Copy link
Collaborator Author

⬆️ @cortinico bump :)

@cortinico
Copy link
Member

LGMT, @cortinico please check if that's not breaking for anybody

Should be safe to remove 👍

@thymikee thymikee merged commit ac284e2 into main Nov 30, 2023
@thymikee thymikee deleted the fix/android-remove-root-option branch November 30, 2023 10:32
@thymikee thymikee changed the title fix(android): remove root argument chore(android): remove root argument Nov 30, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants