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

[WIP] Reboot of CEP7 development #296

Open
wants to merge 4 commits into
base: source
Choose a base branch
from
Open

[WIP] Reboot of CEP7 development #296

wants to merge 4 commits into from

Conversation

gonuke
Copy link
Member

@gonuke gonuke commented Oct 15, 2019

In light of possible expansions and changes to the Cyclus community, I am rebooting the consideration of CEP7.

@gonuke gonuke requested review from jbae11 and katyhuff October 15, 2019 16:05
@jbae11
Copy link
Contributor

jbae11 commented Oct 15, 2019

@gonuke Thank you for putting this together! If I understand correctly, there is now going to be a process to become a Cyclus developer?

@gonuke
Copy link
Member Author

gonuke commented Oct 15, 2019

@jbae11 - This is not about a process to become a developer - we don't need an more obstacles there.

This is about a process to govern a community of those who effectively volunteer their effort to that community.

Copy link
Contributor

@jbae11 jbae11 left a comment

Choose a reason for hiding this comment

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

Thanks!

source/cep/cep7.rst Show resolved Hide resolved
source/cep/cep7.rst Outdated Show resolved Hide resolved
@gonuke gonuke changed the title Reboot of CEP7 development [WIP] Reboot of CEP7 development Oct 16, 2019
Fix typos - thans @jbae11
governance within this organization. Membership in this organization is
governed by a process defined below.

Council of Principal Investigators
Copy link

Choose a reason for hiding this comment

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

Some thoughts about the council:

  • If a community manager exists, their perspective would be really useful in the council. Maybe it would be helpful to reframe this council as a "leadership" council that is encoded to have a set of PIs on it
  • The perspective of a person who is exclusively a user, and not a developer, can be helpful at a top level council making decisions. Would it be helpful to add in a role for this council that fills this representation?

technical grounds.
6. Changes are adopted when at least one member of the Organization has
approved the changes and no members of the Organization are blocking the
changes.
Copy link

Choose a reason for hiding this comment

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

For more substantive changes (new features maybe?) it might be worth adding in a requirement that a PR is open for a period of time (perhaps two weeks?) before merging is allowed. That way community members have time to weigh in even if they aren't able to attend a development meeting where new features are getting pushed forwards.


:CEP: 7
:Title: Cyclus Community Constitution
:Last-Modified: 2015-08-18
Copy link

Choose a reason for hiding this comment

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

If this CEP gets updated as a result of this meeting (or the hackathon) then we should update this (or do we have a bot that handles it?)

Suggested change
:Last-Modified: 2015-08-18
:Last-Modified: 2024-03-21

:CEP: 7
:Title: Cyclus Community Constitution
:Last-Modified: 2015-08-18
:Author: Paul Wilson
Copy link

Choose a reason for hiding this comment

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

Do reviewers count as authors for CEPs?

@katyhuff
Copy link
Member

It appears I've been ignoring this review request for 5 years. Madicken (@munkm ) seems to have done a recent review. Is my review still needed?

@gonuke
Copy link
Member Author

gonuke commented May 23, 2024

Since CEP7 is about governance, I think your input would still be valuable

Comment on lines +132 to +143
Best Practices
---------------

(should these be in their own CEP)

1. Substantive changes to the work of others should require those others to
comment during the review process.
2. Individual pull requests should be as small as possible to facilitate
timely review.
3. Before embarking on long time-intensive changes, it is wise to collect the
opinion of the |Cyclus| community on the value of such changes, generally
through the mailing list.
Copy link

Choose a reason for hiding this comment

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

I like the idea of having a development best practices CEP (I think this is what is commented on in l135?). This section isn't specific to governance and broad decision making, but it is something we want to encourage (or require?) community members to practice.

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