Skip to content

Conversation

BenLorantfy
Copy link
Contributor

Summary:

Adds a note explaining a caveat with auto-linking in a monorepo. Ran into this problem today trying to get a monorepo working.

Adds a note explaining a caveat with auto-linking in a monorepo.  Ran into this problem today trying to get a monorepo working.
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.

Sounds right. @grabbou wanna add anything?

@liamjones
Copy link
Contributor

I believe another option if using yarn workspaces, is specifying the native dependency as 'nohoist', correct? If someone can confirm my belief, it might be worth adding that too?

@thymikee
Copy link
Member

thymikee commented Jan 8, 2021

There's no need for "nohoist" when using yarn workspaces.

Co-authored-by: James George <jamesgeorge998001@gmail.com>
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.

Sounds good, thanks for the contribution!

@thymikee thymikee changed the title Update monorepo section with another note docs: update monorepo autolinking section with a disclaimer Feb 4, 2021
@thymikee thymikee merged commit 35481b2 into react-native-community:master Feb 4, 2021
@adrianha
Copy link
Contributor

adrianha commented Jul 12, 2022

Just opened a PR #1647 that might help to solve the caveat defined here. Please let me know what you think @thymikee @BenLorantfy

updated: fix PR link

@liamjones
Copy link
Contributor

@adrianha Your link is broken, looks like it was meant to point to PR #1647?

@adrianha
Copy link
Contributor

@adrianha Your link is broken, looks like it was meant to point to PR #1647?

Thanks, updated

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.

5 participants