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 pass at our governance model #2

Merged
merged 2 commits into from
Jan 18, 2018
Merged

Conversation

duglin
Copy link
Collaborator

@duglin duglin commented Dec 14, 2017

Signed-off-by: Doug Davis dug@us.ibm.com

@caniszczyk
Copy link
Contributor

LGTM, but I'd also add a MAINTAINERS file that lists the actual maintainers with voting rights

@duglin
Copy link
Collaborator Author

duglin commented Dec 15, 2017

The problem with keeping a MAINTAINERS file up to date is that it might change ever week based on attendance - which will be kind of annoying. I plan on having the spreadsheet automagically calculate who has rights (see https://docs.google.com/spreadsheets/d/1bw5s9sC2ggYyAiGJHEk7xm-q2KG6jyrfBy69ifkdmt0/edit#gid=0 for my working prototype), so the info will be easily available. I guess we could create a MAINTAINERS.md file that just say "go look at the [link] spreadsheet".

Signed-off-by: Doug Davis <dug@us.ibm.com>
@carimura
Copy link

carimura commented Jan 4, 2018

I could write an Fn func that grabs names from spreadsheet and updates MAINTAINERS.md. you know, dogfooding working group projects. :)

* Any motion that does not have "unanimous consent" will result in a formal
vote. See [Voting](#voting).

## PRs
Copy link
Contributor

Choose a reason for hiding this comment

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

Do we need to raise an issue that the PR addresses first?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

I know some groups like to do that, but I think its kind of silly overhead to require an issue. I'm personally ok with someone opening a PR and explaining why they think the change is needed directly in the PR. I think having it all in one spot anyway ;-)

@duglin
Copy link
Collaborator Author

duglin commented Jan 17, 2018

On last week's call @markpeek agreed to suggest some wording around how to deal with multiple people from the same company.

Signed-off-by: Doug Davis <dug@us.ibm.com>
@cloudevents cloudevents deleted a comment from markpeek Jan 18, 2018
@duglin
Copy link
Collaborator Author

duglin commented Jan 18, 2018

Voting section has been updated -please review for today's call.

@duglin
Copy link
Collaborator Author

duglin commented Jan 18, 2018

Per today's call this was approved by unanimous consent

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.

4 participants