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

feat: mention local modules #3877

Merged
merged 8 commits into from
Feb 9, 2024

Conversation

szymonrybczak
Copy link
Contributor

Hey! 👋

I'm adding informations about local modules directive with which user can create native component, native module etc. with one command.

Read more here: react-native-community/discussions-and-proposals#702 (comment)

@szymonrybczak
Copy link
Contributor Author

cc. @satya164 @cortinico @Simek

@netlify
Copy link

netlify bot commented Oct 10, 2023

Deploy Preview for react-native ready!

Name Link
🔨 Latest commit 99bb676
🔍 Latest deploy log https://app.netlify.com/sites/react-native/deploys/65c65e5a5661640008fc878e
😎 Deploy Preview https://deploy-preview-3877--react-native.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@szymonrybczak szymonrybczak force-pushed the feat/mention-local-module branch from 1062c27 to af6b331 Compare October 10, 2023 16:05
@szymonrybczak szymonrybczak force-pushed the feat/mention-local-module branch from af6b331 to 91a4386 Compare October 11, 2023 07:11
@cortinico cortinico requested a review from cipolleschi October 31, 2023 22:08
Comment on lines +60 to +62
```shell
npm install
```
Copy link
Contributor

Choose a reason for hiding this comment

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

Should we make a call and only use yarn, @cortinico?
I remember having discussions that npm is brittle, but I'm, not totally sure.

Copy link
Contributor

Choose a reason for hiding this comment

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

In theory this should be tabbed npm/yarn as we do for all the other pages (like here https://reactnative.dev/docs/react-devtools)

docs/native-modules-intro.md Outdated Show resolved Hide resolved
docs/native-modules-intro.md Outdated Show resolved Hide resolved
szymonrybczak and others added 2 commits November 2, 2023 15:09
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
Co-authored-by: Riccardo Cipolleschi <riccardo.cipolleschi@gmail.com>
docs/local-library-setup.md Outdated Show resolved Hide resolved
docs/local-library-setup.md Outdated Show resolved Hide resolved
Co-authored-by: Satyajit Sahoo <satyajit.happy@gmail.com>
@szymonrybczak
Copy link
Contributor Author

Can we get this merged? :) cc: @cortinico @cipolleschi

Copy link
Collaborator

@Simek Simek left a comment

Choose a reason for hiding this comment

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

👌

Copy link
Contributor

@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 just a minor bit

docs/local-library-setup.md Outdated Show resolved Hide resolved
Co-authored-by: Satyajit Sahoo <satyajit.happy@gmail.com>
@cortinico
Copy link
Contributor

@szymonrybczak let's fix the CI and then we can merge it

@cortinico cortinico merged commit 2c4cb4f into facebook:main Feb 9, 2024
4 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants