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

Move How to Contribute to documentation and update it #7817

Merged
merged 2 commits into from
Sep 27, 2016
Merged

Move How to Contribute to documentation and update it #7817

merged 2 commits into from
Sep 27, 2016

Conversation

gaearon
Copy link
Collaborator

@gaearon gaearon commented Sep 27, 2016

I’m working on adding more sections to Contributing on the website.
I imagine the final structure like this:

  • How to Contribute
  • Design Principles
  • Code Conventions (I want to do this in another PR)
  • Implementation Overview (I want to do this in another PR)

This PR moves most content from CONTRIBUTING.md to a new How to Contribute section on the website, making it more prominent. I also took an opportunity to update some its parts to better reflect our contributing workflow and add some tips.

@vjeux
Copy link
Contributor

vjeux commented Sep 27, 2016

Looks good to me. @spicyj is in PTO right now so if you prefer to move fast, feel free to merge it and I'm sure that Ben will be able to submit feedback after the fact.

@ghost ghost added the CLA Signed label Sep 27, 2016
@gaearon gaearon merged commit c82bcef into facebook:master Sep 27, 2016
@gaearon gaearon deleted the how-to-contribute branch September 27, 2016 22:20
gaearon added a commit that referenced this pull request Sep 27, 2016
* Move How to Contribute to documentation and update it

* Consistent formatting
acusti pushed a commit to brandcast/react that referenced this pull request Mar 15, 2017
* Move How to Contribute to documentation and update it

* Consistent formatting
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.

3 participants