Skip to content

Move captains and committers to another repo - #6396

Closed
UlisesGascon wants to merge 4 commits into
masterfrom
ulises/express-session
Closed

Move captains and committers to another repo#6396
UlisesGascon wants to merge 4 commits into
masterfrom
ulises/express-session

Conversation

@UlisesGascon

@UlisesGasconUlisesGascon commented Mar 16, 2025•
edited
Loading

Copy link
Copy Markdown
Member

NO MERGE BEFORE expressjs/.github#18

This PR reflects that the active repo list and initiatives have been moved to another repository.

@UlisesGasconUlisesGascon self-assigned this Mar 16, 2025
@UlisesGascon
UlisesGascon requested a review from a teamMarch 16, 2025 18:44
@bjohansebas

bjohansebas commented Mar 16, 2025

Copy link
Copy Markdown
Member

Cool, although these changes were previously blocked because we didn't want to maintain that list here since it was unmanageable (see #6087, #6033), maybe the process should even be through creating an issue and waiting, perhaps 7 days, to be accepted or rejected , as was partially done with the committers on the website (see #6057).

I wouldn't want to be unable to get write permissions just because of that small detail of maintaining the list.

UlisesGascon added a commit to expressjs/.github that referenced this pull request Mar 17, 2025
@UlisesGascon
docs: create a file for captains and committers
7e622f7
Ref: expressjs/express#6396 (comment)
Port: expressjs/express#6087
Port: expressjs/express#6033
@UlisesGasconUlisesGascon mentioned this pull request Mar 17, 2025
Closed
@UlisesGascon

UlisesGascon commented Mar 17, 2025

Copy link
Copy Markdown
MemberAuthor

You are absolutely right @bjohansebas. I just updated the PR to reflect this 👍

@UlisesGasconUlisesGascon changed the title docs: add committersMove captains and committers to another repoMar 17, 2025
@UlisesGasconUlisesGascon mentioned this pull request Mar 17, 2025
Closed
UlisesGascon
UlisesGascon commented May 11, 2025
View reviewed changes
Comment threadContributing.md Outdated
UlisesGascon
UlisesGascon commented May 11, 2025
View reviewed changes
Comment threadContributing.md Outdated
@UlisesGascon

UlisesGascon commented May 11, 2025

Copy link
Copy Markdown
MemberAuthor

This is now ready to be merged, I updated the links 👍

@bjohansebas
bjohansebas deleted the ulises/express-session branch July 14, 2025 02:13
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@Phillip9587Phillip9587Phillip9587 approved these changes
@bjohansebasbjohansebasbjohansebas approved these changes
@carpassecarpasseAwaiting requested review from carpasse

Assignees

@UlisesGasconUlisesGascon

Labels

meta

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@UlisesGascon@bjohansebas@Phillip9587