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

Update dev readme some more #825

Merged
merged 1 commit into from
Apr 19, 2017
Merged

Update dev readme some more #825

merged 1 commit into from
Apr 19, 2017

Conversation

0x80
Copy link
Contributor

@0x80 0x80 commented Apr 19, 2017

  • Remove mentions of yarn
  • Cross-reference contributing and gatsby-dev-cli readme files

@@ -35,6 +31,8 @@ this program running.
Typically you'll also want to run `npm run watch` in the Gatsby repo to set up
watchers to build Gatsby source code.

More detailed instruction for setting up your Gatsby repository are found [here](/CONTRIBUTING.md).
Copy link
Contributor

Choose a reason for hiding this comment

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

How about you just link to https://www.gatsbyjs.org/docs/how-to-contribute/ (which is just CONTRIBUTING.md)

Copy link
Contributor

Choose a reason for hiding this comment

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

Also how about changing the language to "setting up your Gatsby development environment"

@gatsbybot
Copy link
Collaborator

gatsbybot commented Apr 19, 2017

Deploy preview ready!

Built with commit 6f9ee62

https://deploy-preview-825--gatsbygram.netlify.com

* Remove mentions of yarn
* Cross-reference contributing and gatsby-dev-cli readme files
@KyleAMathews
Copy link
Contributor

Thanks!

@KyleAMathews KyleAMathews merged commit 7da5ef4 into gatsbyjs:1.0 Apr 19, 2017
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