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

Add initial governance details #1

Merged
merged 26 commits into from
Jul 14, 2019
Merged

Add initial governance details #1

merged 26 commits into from
Jul 14, 2019

Conversation

TomWFox
Copy link
Contributor

@TomWFox TomWFox commented Jun 26, 2019

This is the proposed governance document for the entirety of the Parse Platform. It follows the Meritocratic Governance Model. I have made some minor adjustments such as removing references to a mailing list and making the document a bit more concise in places.

@TomWFox
Copy link
Contributor Author

TomWFox commented Jun 26, 2019

I feel like this is in a pretty fit state now... I would like to add more information about the voting process etc as we find what works best.

For the most part we are already following this model but I will just highlight some areas I feel we do not currently conform to bring these to everyone's attention...

  • We don't wait 72 hours before merging PRs as a policy
  • We don't have a process for committers proposing new committers (people with write access) & we don't vote on said proposals
  • We don't have a formal PMC (project management committee) or PMC Chair
  • We don't have a process for voting in or out a PMC Chair
  • Non-sensitive management/planning discussion isn't always public
  • We don't vote when a lazy consensus on a PR is not reached (although I like almost all of the time a consensus is reached so I don't think this is a major issue atm)

@TomWFox TomWFox marked this pull request as ready for review June 26, 2019 20:46
@TomWFox TomWFox requested a review from davimacedo June 27, 2019 11:01
Copy link
Contributor

@acinader acinader left a comment

Choose a reason for hiding this comment

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

I'm gonna make all the obvious corrections now....

CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
README.md Show resolved Hide resolved
acinader and others added 4 commits June 27, 2019 15:38
* Some simple word simthery suggestions

I didn't go full oxford comma.

* Fun with prepositions.
@TomWFox TomWFox requested a review from acinader June 27, 2019 16:01
README.md Outdated Show resolved Hide resolved
@davimacedo
Copy link
Member

Should we include in README (or elsewhere) who are the current commiters and PMCs ?

@TomWFox
Copy link
Contributor Author

TomWFox commented Jun 27, 2019

@davimacedo definitely, I'd like to try to keep the README as short as possible so I'm tempted to put it in a separate doc, what would be your preference? (@acinader any preference?)

@acinader
Copy link
Contributor

thanks for asking. my preference is that you decide ;)

@davimacedo
Copy link
Member

It can be a separate document. I just think it is important to keep track of the chairs so anyone can check if the approvals were given by the right people (and sufficient amount) at that time.

@TomWFox
Copy link
Contributor Author

TomWFox commented Jun 27, 2019

Ok TEAM.md added, I only included people who are at least vaguely active

@TomWFox
Copy link
Contributor Author

TomWFox commented Jul 2, 2019

@acinader @davimacedo I've added the info about voting now - can you take a look? (I think it should be done after this 😃)

@TomWFox TomWFox requested a review from davimacedo July 2, 2019 13:04
Copy link
Member

@davimacedo davimacedo left a comment

Choose a reason for hiding this comment

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

It seems ok for me. I am just sending some typos I found.

VOTING.md Outdated Show resolved Hide resolved
VOTING.md Show resolved Hide resolved
VOTING.md Outdated Show resolved Hide resolved
CONTRIBUTING.md Outdated Show resolved Hide resolved
PMC -> Project Management Committee
Copy link
Contributor

@acinader acinader left a comment

Choose a reason for hiding this comment

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

this looks good to me. Thanks @TomWFox

@TomWFox TomWFox merged commit 454e71d into master Jul 14, 2019
@TomWFox TomWFox deleted the TomWFox-patch-1 branch July 14, 2019 19:01
@TomWFox
Copy link
Contributor Author

TomWFox commented Jul 14, 2019

The proposal was successful so I've merged this.

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