-
Notifications
You must be signed in to change notification settings - Fork 49.2k
[Gatsby] README.md instructions #10945
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
Conversation
www/README.md
Outdated
### Running locally | ||
|
||
1. `yarn dev` to build the site with Gatsby, a (React-based!) static site generator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
perhaps change to "to start the hot-reloading development server powered by Gatsby"?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure, and drop the a (React-based!) static site generator
?
www/README.md
Outdated
|
||
1. `cd www` to go into the website project | ||
1. `yarn` to install the website's NPM dependencies |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
You also need to run yarn at the root right? For Prettier, etc?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good spot, thanks
www/README.md
Outdated
1. Node: install version 7.6 or greater | ||
1. Yarn: `npm i -g yarn` to install it globally via NPM | ||
1. A clone of the [https://github.com/facebook/react](React repo) on your local machine |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Wrong order in Markdown link (doesn't render)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Doh, fixed :)
www/README.md
Outdated
1. `git add -A && git commit -m "My message"` (replacing `My message` with a commit message, such as `Fixed header logo on Android`) to stage and commit your changes | ||
1. `git push yourname wwwTheNameOfMyChange` | ||
1. Go to the [https://github.com/facebook/react](React repo) and you should see recently pushed branches. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Same.
www/README.md
Outdated
1. Follow the "Running locally" instructions | ||
1. Save the files and check in the browser (any React components in `src/components` or `src/templates`) will hot-reload) | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Worth noting that if you change site internals like plugins, it may be necessary to remove .cache
directory and restart the server for changes to take effect.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good idea. Split these out into different procedures
Deploy preview ready! Built with commit 6ee9c67 |
www/README.md
Outdated
1. Save the files and check in the browser | ||
1. Any React components in `src/components` or `src/templates`) will hot-reload | ||
1. Some other JS files, such as `html.js`, may require a full reload |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Changing html.js triggers a page refresh automatically so this line isn't necessary
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Oh yeah! Never noticed that before 👍
1. Follow the "Running locally" instructions | ||
1. Save the files and check in the browser | ||
1. Any React components in `src/components` or `src/templates` will hot-reload |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Had one more suggestion :-)
"Edits to markdown files in /docs
and to React components in…"
Because the new React-based docs are a great way for people to contribute, I figured a more thorough README.md could be useful.
What do you think?
cc @bvaughn @gaearon