Skip to content

Allow team leads posting Inside Rust blog posts #670

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

Merged
merged 1 commit into from
Aug 14, 2020

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
Member

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