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

GOV add a documentation team #26714

Merged
merged 2 commits into from
Aug 12, 2023
Merged

Conversation

adrinjalali
Copy link
Member

This PR proposes adding a documentation team to the governance model.

The initial members are proposed to be @ArturoAmorQ and @lucyleeow

Very happy to get feedback on the wording of the description of the team.

The team is already created on GH, but has no extra permissions till we merge this PR. I will call for a vote once an initial round of discussions are concluded here.

@github-actions
Copy link

github-actions bot commented Jun 27, 2023

✔️ Linting Passed

All linting checks passed. Your pull request is in excellent shape! ☀️

Generated for commit: d52844d. Link to the linter CI: here

Copy link
Member

@jjerphan jjerphan left a comment

Choose a reason for hiding this comment

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

LGTM.

Thank you, @adrinjalali.

I think we can be liberal and trust people for making decision while recognizing their work and involvement in the project.

Here are some minor nitpicks.

doc/about.rst Outdated Show resolved Hide resolved
doc/governance.rst Outdated Show resolved Hide resolved
doc/governance.rst Outdated Show resolved Hide resolved
Copy link
Member

@glemaitre glemaitre left a comment

Choose a reason for hiding this comment

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

+1

Copy link
Member Author

@adrinjalali adrinjalali left a comment

Choose a reason for hiding this comment

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

I think we can be liberal and trust people for making decision while recognizing their work and involvement in the project.

@jjerphan a governance document is a contract, and better be clear about peoples rights and responsibilities than not. So I rather not remove the part about the team being able to merge only doc PRs.

doc/governance.rst Outdated Show resolved Hide resolved
@thomasjpfan
Copy link
Member

A vote was called on the mailing list. The voting period is one month and will conclude on August 10th. According to our governance docs, no changes can be made to this PR during the voting period.

@GaelVaroquaux
Copy link
Member

@agramfort : if I am not wrong, you have not had a look at this. If you are find with this change, would you mind approving it.

@GaelVaroquaux
Copy link
Member

Trying to solicit more approvals or discussion, @scikit-learn/core-devs : please approve this PR if you are fine with this change of our governance. It adds a documentation team that helps reviewing and merge documentation, in an effort to adapt to our broader team.

@glemaitre
Copy link
Member

We should be able to merge this one since the 1 month voting period is passed?

@adrinjalali
Copy link
Member Author

Merging, and will give the @scikit-learn/documentation-team merge rights. Excited! I'll also send an email on our mailing list.

@adrinjalali adrinjalali merged commit 622d546 into scikit-learn:main Aug 12, 2023
@adrinjalali adrinjalali deleted the documentation branch August 12, 2023 09:39
@GaelVaroquaux
Copy link
Member

GaelVaroquaux commented Aug 12, 2023 via email

TamaraAtanasoska pushed a commit to TamaraAtanasoska/scikit-learn that referenced this pull request Aug 21, 2023
akaashpatelmns pushed a commit to akaashp2000/scikit-learn that referenced this pull request Aug 25, 2023
glemaitre pushed a commit to glemaitre/scikit-learn that referenced this pull request Sep 18, 2023
jeremiedbb pushed a commit that referenced this pull request Sep 20, 2023
REDVM pushed a commit to REDVM/scikit-learn that referenced this pull request Nov 16, 2023
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.