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

simplify-governance #67

Merged
merged 7 commits into from
Oct 14, 2020
Merged

Conversation

lehnberg
Copy link
Contributor

@lehnberg lehnberg commented Sep 19, 2020

Rendered text


Summary

This governance iteration replaces the previous process that was set out in RFC#0002 with a simplified and better defined version. Specifically it:

  • defines its remit to be around the repos, projects, and communities that are centered around the /mimblewimble GitHub organization;
  • reverts the core team to the original technocratic council structure, whose sole responsibility is managing the general fund; and
  • explicitly enables individual contributors and teams to self-organize for other aspects of decision making.

joint work with @Paouky

@lehnberg lehnberg added the core Related to core team label Sep 22, 2020
@lehnberg
Copy link
Contributor Author

Assigned @quentinlesceller as shepherd

Copy link
Member

@quentinlesceller quentinlesceller left a comment

Choose a reason for hiding this comment

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

Two minor comments below:

text/0000-simplify-governance.md Outdated Show resolved Hide resolved
text/0000-simplify-governance.md Outdated Show resolved Hide resolved
@quentinlesceller
Copy link
Member

In line with our governance process, this RFC can be considered being in Final Comment Period, with a disposition to merge in two weeks time, on October 13.

Please ensure any comments are made before then!

@quentinlesceller quentinlesceller added the in FCP Currently in Final Comment Period label Sep 29, 2020
@MCM-Mike
Copy link

MCM-Mike commented Oct 2, 2020

We we should use the pull-request for comments I would suggest the following changes to the proposed RFC

Under section Community-level explanation -> Teams sub section -> Other teams to strike out (delete) the "security team"

Why?
Based on my understanding is the security team should not be a self manged team without any core members being part of it. This could have some serious security implications, like in the following example, when we receive a disclosure let say for our CLI Grin-Wallet . At least one core-team member should be informed as well, if there is a serious thread to GRIN.

I do know the team is managed by John, a responsible and hopefully long-term trustworthy person, but I would suggest this security team has at least one core-member who will be informed at any times there is an security issue with GRIN.

This suggestion is again a more centralized approach, but sometimes in IT-Security it is needed to keep GRIN safe.

@DavidBurkett
Copy link
Contributor

@MCM-Mike The assumption you're making is that only council members can be trusted (P.S. there is no "core"). There's no reason to think that the contributors that do the most, and are the most trusted, will even be on the council. The council only exists to deal with funding, nothing else. To require them to be involved with security makes no sense. That's like requiring the finance department at a corporation to be involved with IT security.

Copy link
Contributor

@DavidBurkett DavidBurkett 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 great. Thanks for putting this together @Paouky and @lehnberg

@lehnberg
Copy link
Contributor Author

lehnberg commented Oct 5, 2020

@MCM-Mike yes, just to add to what @DavidBurkett writes above:

  • there would be no core team if this RFC gets merged.
  • the security team is already self-managed. It currently consists of myself and @j01tz (so you have two existing core team members, or 100% of the team constituting of core members at the moment), and we run our own show without the involvement of others. This would continue.

@quentinlesceller quentinlesceller merged commit 7313ae7 into mimblewimble:master Oct 14, 2020
@quentinlesceller
Copy link
Member

🎉 Wohooo! This RFC has now been merged!

@lehnberg
Copy link
Contributor Author

Goodbye, Core team. 👋
Hello council, nice to see you again. 😎

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
core Related to core team in FCP Currently in Final Comment Period
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants