Skip to content

docs: update GOVERNANCE.md to clarify repo captain permissions - #363

Merged
jonchurch merged 3 commits into
expressjs:governance-updatesfrom
bjohansebas:bjohansebas-patch-1
Jan 6, 2026
Merged

docs: update GOVERNANCE.md to clarify repo captain permissions#363
jonchurch merged 3 commits into
expressjs:governance-updatesfrom
bjohansebas:bjohansebas-patch-1

Conversation

@bjohansebas

Copy link
Copy Markdown
Member

This came up in #326, to clarify a bit the permissions of captains.

CopilotAI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Pull Request Overview

This PR aims to clarify the permissions and roles of repo captains as discussed in the Express discussions.

  • Clarifies the scope of repo captain permissions
  • Updates wording to reflect the intended repository maintainership role

Comment threaddocs/GOVERNANCE.md Outdated
Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
@bjohansebas

Copy link
Copy Markdown
MemberAuthor

cc: @expressjs/express-tc

@bjohansebas
bjohansebas requested a review from a teamJuly 14, 2025 02:14
@bjohansebasbjohansebas changed the title docs: update Contributing.md to clarify repo captain permissionsdocs: update GOVERNANCE.md to clarify repo captain permissionsAug 8, 2025
@github-actionsgithub-actionsBot mentioned this pull request Sep 9, 2025
Comment threaddocs/GOVERNANCE.md Outdated
Co-authored-by: Jon Church <me@jonchurch.com>
@wesleytodd

Copy link
Copy Markdown
Member

Blocked on (similar to #375): openjs-foundation/cross-project-council#1607

@sheplusheplu left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@jonchurch

Copy link
Copy Markdown
Member

I agree with the spirit fo the PR that captains hsoudl have admin and publishh on their repos.

Approved int hat spirit, idk if there is dissent here still, would love to see other @expressjs/express-tc opinions via approve/reqquest change

@jonchurch

Copy link
Copy Markdown
Member

Governance docs can have this detail in it, I don't think we are blocked on the CHarter vs Governance issue anymore cc @wesleytodd

@wesleytodd
wesleytodd changed the base branch from master to governance-updatesDecember 8, 2025 19:48
@wesleytodd

Copy link
Copy Markdown
Member

Changed target branch to governance-updates so that we can land these and then take one consolidated branch to the CPC for approval.

@jonchurch
jonchurch merged commit d7b3937 into expressjs:governance-updatesJan 6, 2026
1 check passed
@bjohansebas
bjohansebas deleted the bjohansebas-patch-1 branch January 6, 2026 22:57
Sign up for freeto 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.

6 participants

@bjohansebas@wesleytodd@jonchurch@sheplu@UlisesGascon