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

Allow education to merge image PRs and web presence to control the website source #5340

Merged
merged 4 commits into from
Dec 11, 2024

Conversation

judithpatudith
Copy link
Contributor

Adding education and web presence to the code owners file for pieces of the repo that they will need to merge PRs to.

@judithpatudith judithpatudith requested a review from a team as a code owner December 6, 2024 19:42
johanbrandhorst
johanbrandhorst previously approved these changes Dec 6, 2024
Copy link
Collaborator

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

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

This change will potentially block some merges to the enterprise repo without approval from this team, is there a channel where we can prompt for reviews from this team when necessary?

@judithpatudith
Copy link
Contributor Author

@johanbrandhorst Sorry, that definitely wasn't the intention, good catch! I think this recent change keeps the Boundary team's global permissions. My hope is to let web approve website PRs and the education team approve docs PRs. We don't need to require their approvals specifically.

johanbrandhorst
johanbrandhorst previously approved these changes Dec 7, 2024
Copy link
Collaborator

@johanbrandhorst johanbrandhorst left a comment

Choose a reason for hiding this comment

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

Great, thank you! I think we can backport this to 0.16-18 as well

@judithpatudith
Copy link
Contributor Author

sorry for the back and forth. This last change lets us edit the docs side navigation as well

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.

5 participants