Skip to content

MNT: update github access role based on project document#2877

Merged
tech3371 merged 2 commits intoIMAP-Science-Operations-Center:devfrom
tech3371:git_access_update
Mar 30, 2026
Merged

MNT: update github access role based on project document#2877
tech3371 merged 2 commits intoIMAP-Science-Operations-Center:devfrom
tech3371:git_access_update

Conversation

@tech3371
Copy link
Copy Markdown
Contributor

Change Summary

Overview

Update based on project document and yesterday findings.

File changes

Testing

@tech3371
Copy link
Copy Markdown
Contributor Author

Added all people who will need to be informed of what access we need to give if new contributor comes. Right now, no action needed from SDC.

Copy link
Copy Markdown
Collaborator

@jtniehof jtniehof left a comment

Choose a reason for hiding this comment

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

I like that this is a lot more focused on the specifics of what we care about. Only suggestion would be to consider adding a link to https://docs.github.com/en/organizations/managing-user-access-to-your-organizations-repositories/managing-repository-roles/repository-roles-for-an-organization#permissions-for-each-role so that it's easy to get at the full details if someone needs to.

@tech3371
Copy link
Copy Markdown
Contributor Author

I like that this is a lot more focused on the specifics of what we care about. Only suggestion would be to consider adding a link to https://docs.github.com/en/organizations/managing-user-access-to-your-organizations-repositories/managing-repository-roles/repository-roles-for-an-organization#permissions-for-each-role so that it's easy to get at the full details if someone needs to.

Totally! I will link that in that page!

Copy link
Copy Markdown
Contributor

@subagonsouth subagonsouth left a comment

Choose a reason for hiding this comment

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

I don't really understand the removal roles, but LGTM.

@tech3371 tech3371 merged commit b01ad61 into IMAP-Science-Operations-Center:dev Mar 30, 2026
14 checks passed
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.

3 participants