Skip to content

Conversation

pietroalbini
Copy link
Member

This PR allows any team lead to approve and merge blog posts for Inside Rust!

r? @nikomatsakis
cc @rust-lang/leads
Closes rust-lang/core-team#18

@pietroalbini pietroalbini merged commit 81c5195 into rust-lang:master Aug 14, 2020
@pietroalbini pietroalbini deleted the codeowners branch August 14, 2020 11:43
Copy link
Member

@jyn514 jyn514 left a comment

Choose a reason for hiding this comment

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

It looks like this removes permissions for @rust-lang/website, was that intentional? (Or is everyone on that team a team lead?)

@XAMPPRocky
Copy link
Contributor

Well I'm not a team lead, so currently I cannot merge. I think it would still be useful to have the website team on it to be able to correct typos, incorrect metadata, and to merge posts that are approved by working group/project group leads. Though I also don't mind not doing that work, and having that shared amongst the leads as well.

@jyn514
Copy link
Member

jyn514 commented Aug 14, 2020

I think doing the work should be separate from being able to approve and merge it. Currently if a team lead fixes a typo, you wouldn't be able to merge it which doesn't seem right.

@pietroalbini
Copy link
Member Author

@jyn514 good point! Reopened rust-lang/core-team#18.

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.

permissions on blog.rust-lang.org
5 participants