Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

yashLadha
Copy link
Contributor

No description provided.

@nodejs-github-bot nodejs-github-bot added c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels Mar 30, 2021
@PoojaDurgad PoojaDurgad added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Apr 5, 2021
jasnell
jasnell approved these changes Apr 6, 2021
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@jasnell jasnell removed the needs-ci PRs that need a full CI run. label Apr 7, 2021
@nodejs-github-bot
Copy link
Collaborator

@yashLadha
Copy link
Contributor Author

Should I go ahead and deploy, it doesn't seem that big of a change @jasnell 😅 and is open for quite some time.

@jasnell
Copy link
Member

jasnell commented Apr 8, 2021

We have to get a good ci run first. It's irritating that ci has been so flaky but it's gotta be yellow or green before this can land

@nodejs-github-bot
Copy link
Collaborator

RaisinTen pushed a commit that referenced this pull request Apr 8, 2021
PR-URL: #37974
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Pooja D P <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@RaisinTen
Copy link
Member

Landed in 8d63aa8

@RaisinTen RaisinTen closed this Apr 8, 2021
targos pushed a commit that referenced this pull request May 1, 2021
PR-URL: #37974
Reviewed-By: Colin Ihrig <[email protected]>
Reviewed-By: Anna Henningsen <[email protected]>
Reviewed-By: Darshan Sen <[email protected]>
Reviewed-By: Rich Trott <[email protected]>
Reviewed-By: Pooja D P <[email protected]>
Reviewed-By: James M Snell <[email protected]>
@danielleadams danielleadams mentioned this pull request May 3, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. c++ Issues and PRs that require attention from people who are familiar with C++.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants