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

Changed "core contributors" to "top contributors" #1

Merged
merged 2 commits into from
Jul 31, 2018

Conversation

axe-fb
Copy link
Collaborator

@axe-fb axe-fb commented Jul 23, 2018

I think the term "top contributors" may be more representative. The core members are just folks who have been contributing to the framework, and by calling them top contributors, we show how we could someone could become a part of the "core" group.

I think the term "top contributors" may be more representative. The core members are just folks who have been contributing to the framework, and by calling them top contributors, we show how we could someone could become a part of the "core" group.
@turnrye
Copy link
Member

turnrye commented Jul 23, 2018

Hey @axe-fb, thanks for the PR! This was a discussion point that we had, and your change is a good way to help show the path.

README.md Outdated
@@ -26,7 +26,7 @@ It is inspired by the [React RFC repo](https://github.com/reactjs/rfcs), but tod

### Core Meetings

The core team regularly meets to discuss ongoing problems with the goal of drafting solution proposals. The [core meetings](core-meetings/README.md) directory contains meeting notes and topics for this process.
The top contributors regularly meets to discuss ongoing problems with the goal of drafting solution proposals. The [core meetings](core-meetings/README.md) directory contains meeting notes and topics for this process.
Copy link
Member

Choose a reason for hiding this comment

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

"meets" should be "meet"

@turnrye turnrye self-assigned this Jul 23, 2018
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.

None yet

3 participants