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

PEP 8002: Add information about Project Jupyter governance #770

Merged
merged 2 commits into from
Sep 11, 2018

Conversation

willingc
Copy link
Contributor

No description provided.

Copy link
Member

@pitrou pitrou left a comment

Choose a reason for hiding this comment

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

Thanks @willingc ! You didn't need a review to merge this (other people have pushed directly to the peps repo), but since you asked I posted a few comments. Very insightful overall.

pep-8002.rst Outdated
Core contributors are individuals who are given rights, such as commit privileges,
to act in the best interest of the project within their area of expertise or
subproject. An existing core contributor typically recommends someone be given
core contributor rights by gathering consensus from project leads.
Copy link
Member

Choose a reason for hiding this comment

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

Who are the project leads? Existing core contributors, or the Steering Council?

pep-8002.rst Outdated
release of a subproject.

There is a weekly, public Project-wide meeting that is recorded and posted on
YouTube. Some larger GitHub organizations, e.g. JupyterLab and JupyterHub, may
Copy link
Member

Choose a reason for hiding this comment

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

Which, I presume, are Jupyter sub-projects or sub-teams?

@willingc willingc merged commit b930b50 into python:master Sep 11, 2018
@willingc willingc deleted the add-jupyter branch September 11, 2018 13:37
@willingc
Copy link
Contributor Author

Thanks @pitrou for the comments and review. I've added additional clarification on the items that you suggested.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants