Skip to content

Update docs #830

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

Merged
merged 13 commits into from
Nov 4, 2017
Merged

Update docs #830

merged 13 commits into from
Nov 4, 2017

Conversation

BookOfGreg
Copy link
Member

@BookOfGreg BookOfGreg commented Nov 4, 2017

@rmosolgo In this PR I change a lot of the Readme to try make it easier to follow the main sections and display the main benefits at the top of the README.
Also added Contributing and Code of Conduct as they are standard and can encourage best practice.

Small wording tweak in the Issues Template, it would be good to review it as almost all issues that come in are of the form "- [x] Discussion, something is wrong" and reproducing is hard.

I want to continue to add to the Wiki more How to's as I've been able to use the react-rails-example-app to close a good dozen issues where folk have just needed guidance. The issues I marked as Discussion are good candidates to close when we have a matching Wiki entry for them.

Also mentioned @renchap 's Webpacker-react in the related projects as it looks promising.

Thoughts?
Closes #825

@rmosolgo
Copy link
Member

rmosolgo commented Nov 4, 2017

👍 Looks like a nice reorg! I appreciate putting the development instructions into CONTRIBUTING, that makes the readme a bit less intimidating 😆 Agreed about the issue template, I added it a while ago but I bet if it were revised, we could get more value out of it.

Borrow content from Rails' Issue template. It emphasizes reproduction.
@BookOfGreg BookOfGreg merged commit b3fac2c into reactjs:master Nov 4, 2017
@BookOfGreg BookOfGreg deleted the update-docs branch November 4, 2017 13:31
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.

TypeError: React.PropTypes is undefined. Migrating 2.3 -> 2.4
2 participants