Skip to content

Conversation

@bjohansebas
Copy link
Member

As discussed in the recent TC meeting, it is important to clarify what to do as a triager regarding security matters.

cc: @expressjs/express-tc @expressjs/triagers

Copy link
Member

@UlisesGascon UlisesGascon left a comment

Choose a reason for hiding this comment

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

LGTM! I think that this covers what we had discussed in our last meeting, also the triage team has limit rights to delete issues, so this is the best approach (IMO)

@UlisesGascon UlisesGascon requested review from a team December 13, 2024 14:08
Copy link
Member

@mertcanaltin mertcanaltin left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link

@Sushmeet Sushmeet left a comment

Choose a reason for hiding this comment

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

LGTM

Copy link
Contributor

@carpasse carpasse left a comment

Choose a reason for hiding this comment

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

LGTM

@UlisesGascon
Copy link
Member

As this was already discussed in a TC Meeting and it is aligned with that discussion, I will merge the PR 👍

@UlisesGascon UlisesGascon merged commit 43020ff into expressjs:master Dec 20, 2024
2 checks passed
@bjohansebas bjohansebas deleted the triage-and-security branch December 20, 2024 19:22
@bjohansebas bjohansebas mentioned this pull request Mar 15, 2025
@UlisesGascon UlisesGascon mentioned this pull request Mar 23, 2025
68 tasks
@wesleytodd wesleytodd mentioned this pull request Mar 28, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

6 participants