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

✨ add @ctcpip to CPC members #1312

Merged
merged 1 commit into from
Jun 3, 2024
Merged

Conversation

ctcpip
Copy link
Member

@ctcpip ctcpip commented May 31, 2024

πŸ‘‹

folks looking at this PR probably know me from:

  • CPC meetings
  • OpenJS standards meetings
  • OpenJS security meetings
  • elsewhere?

πŸ™

@ctcpip ctcpip requested a review from a team as a code owner May 31, 2024 20:46
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.

Oh yeah! πŸ₯³

@WaleedAshraf WaleedAshraf changed the title ✨ update CPC members ✨ add @ctcpip to CPC members Jun 3, 2024
@WaleedAshraf WaleedAshraf merged commit 0a1af4a into openjs-foundation:main Jun 3, 2024
1 check passed
@ljharb
Copy link
Member

ljharb commented Jun 3, 2024

@WaleedAshraf this was merged too soon; the governance doc says non-fast-tracked PRs must be open for 72 hours.

@WaleedAshraf
Copy link
Member

@ljharb Ops. Sorry, I thought it was 48 hours. Would you like me to revert it?

WaleedAshraf added a commit that referenced this pull request Jun 3, 2024
@ljharb
Copy link
Member

ljharb commented Jun 3, 2024

No, let's leave it here unless someone objects.

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.

6 participants