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

Add education and web presence ability to approve PRs to website files #1068

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

judithpatudith
Copy link

I'm trying to standardize the education and web presence team's permissions across all the code repos that contain docs. Please let me know if you have any feedback!

@judithpatudith judithpatudith requested a review from a team as a code owner December 13, 2024 21:55
Copy link
Member

@austinvalle austinvalle left a comment

Choose a reason for hiding this comment

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

Hey @judithpatudith 👋🏻 , this all looks good to me, although @hashicorp/web-presence will need to request write access to this repo (I think that's requested via Doormat now?)

image

The other team looks like it already has it 👍🏻

@austinvalle
Copy link
Member

austinvalle commented Dec 13, 2024

While you're requesting that, you'll probably also want to request access to:

Which also have website directories. Once that's done we can update the CODEOWNERS there as well 👍🏻

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.

2 participants