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

Update DEPs 1, 10, and 12 to reflect current governance #81

Merged
merged 6 commits into from
Oct 30, 2023

Conversation

jacobian
Copy link
Member

This updates DEPs 1, 10, and 12 to reflect current governance (will fix #80):

  • Marks DEPs 10 and 12 as final
  • Adds notes to the tops of those DEPs pointing to the Steering Council docs, as reading the DEPs
    to understand how the Steering Council operates is tricky (the DEPs are diffs basically not the
    full info).
  • Updates DEP 1 to reflect those changes, including renaming the Technical Board to Steering
    Council, removing references to "Core Developers" and updating the Shepherd role since core devs
    no longer exist.

As much as possible, this is just a housekeeping cleanup. That's true of first two changes, which
are not substantive, but the last one kinda is. The DEP process as written describes roles that no
longer exist, so I've had to take some license on updating to reflect current roles.

I've tried to keep the changes as minimal and non-consequential as possible, so we can hopefully
get this document back to describing a process that can actually exist. So it's my opinion that
this doesn't need a formal DEP vote, just a rough "looks OK" from the community. But I do plan on
giving the Steering Council a heads-up about this in case they disagree.

Longer term, I think there's a good argument that we need to refresh DEP 1 more holistically (see
#28, for example), but I think that deserves a real conversation. I didn't want to wait on that
conversation to update these docs to make sense with today's structure, hence this PR.

- Marks DEPs 10 and 12 as final
- Adds notes to the tops of those DEPs pointing to the Steering Council
  docs, as reading the DEPs to understand how the Steering Council
  operates is tricky (the DEPs are diffs basically not the full info).
- Updates DEP 1 to reflect those changes, including renaming
  the Technical Board to Steering Council, removing references
  to "Core Developers" and updating the Shepherd role since core
  devs no longer exist.
Copy link
Member

@carltongibson carltongibson left a comment

Choose a reason for hiding this comment

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

Looks good. Thanks @jacobian. Just a few things I noticed.

final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
Copy link
Member

@thibaudcolas thibaudcolas left a comment

Choose a reason for hiding this comment

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

🌈

final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
Copy link
Contributor

@nessita nessita left a comment

Choose a reason for hiding this comment

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

Thank you for this improvements, they are definitely needed and quite helpful!

final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
final/0001-dep-process.rst Outdated Show resolved Hide resolved
@jacobian
Copy link
Member Author

OK, with no objections from anyone on the SC (thread]), I'm gonna merge.

@thibaudcolas
Copy link
Member

Follow-up with similar updates to DEP 7: #91

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.

Document the current DEP process
5 participants