Skip to content

Added npx prefix to react-native commands within docs#1636

Merged
nearestnabors merged 1 commit intofacebook:masterfrom
cepem:consistent-rn-commands
Feb 17, 2020
Merged

Added npx prefix to react-native commands within docs#1636
nearestnabors merged 1 commit intofacebook:masterfrom
cepem:consistent-rn-commands

Conversation

@cepem
Copy link
Copy Markdown
Contributor

@cepem cepem commented Feb 17, 2020

I noticed that in the docs the 'react-native' command sometimes doesn't have the 'npx' prefix and sometimes it does.
Considering that we're using it with 'npx' on the front page, I believe it would be a good idea to stay consistent through all the docs with it.

@cepem cepem requested a review from rickhanlonii February 17, 2020 03:24
@react-native-bot
Copy link
Copy Markdown

Deploy preview for react-native ready!

Built with commit 80cee32

https://deploy-preview-1636--react-native.netlify.com

Changes to docs/ are reflected in the next "master" version.

Thank you for your contributions.

How to ContributeDocumentation Sources

@cepem cepem requested a review from nearestnabors February 17, 2020 03:53
Copy link
Copy Markdown
Contributor

@nearestnabors nearestnabors left a comment

Choose a reason for hiding this comment

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

This makes sense! Thank you for your attention to detail!

Copy link
Copy Markdown
Contributor

@nearestnabors nearestnabors left a comment

Choose a reason for hiding this comment

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

Woops, meant to approve this. Congrats on your first contribution! Hope to see you here in the future!

@nearestnabors nearestnabors merged commit 815fe10 into facebook:master Feb 17, 2020
kaioduarte pushed a commit to kaioduarte/react-native-website that referenced this pull request Feb 24, 2020
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