Skip to content

Conversation

@Qard
Copy link
Member

@Qard Qard commented Sep 2, 2021

PR-URL: #39890
Reviewed-By: James M Snell jasnell@gmail.com
Reviewed-By: Vladimir de Turckheim vlad2t@hotmail.com
Reviewed-By: Gerhard Stöbich deb2001-github@yahoo.de
Reviewed-By: Minwoo Jung nodecorelab@gmail.com
Reviewed-By: Zijian Liu lxxyxzj@gmail.com

PR-URL: nodejs#39890
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Vladimir de Turckheim <vlad2t@hotmail.com>
Reviewed-By: Gerhard Stöbich <deb2001-github@yahoo.de>
Reviewed-By: Minwoo Jung <nodecorelab@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
@Qard Qard added async_hooks Issues and PRs related to the async hooks subsystem. v14.x labels Sep 2, 2021
@github-actions github-actions bot added the needs-ci PRs that need a full CI run. label Sep 2, 2021
@Qard Qard added request-ci Add this label to start a Jenkins CI on a PR. and removed needs-ci PRs that need a full CI run. labels Sep 2, 2021
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2021
@nodejs-github-bot
Copy link
Collaborator

@targos
Copy link
Member

targos commented Sep 2, 2021

The commit lands cleanly on my side and all tests pass. Did you change something to open this PR?

@Qard
Copy link
Member Author

Qard commented Sep 2, 2021

Ah, nope. It lands cleanly.

@targos
Copy link
Member

targos commented Sep 3, 2021

Okay, I pushed it in 5a42be9.

If you want to make sure a change lands on LTS, it should be sufficient to add the lts-watch-vx label and a comment on the original PR.

@targos targos closed this Sep 3, 2021
@Qard
Copy link
Member Author

Qard commented Sep 3, 2021

Ah, good to know, thanks! Sorry, I'm still learning how all the backporting stuff works. 😅

@Qard Qard deleted the backport-39890-to-14 branch September 3, 2021 14:55
@targos
Copy link
Member

targos commented Sep 4, 2021

No worries. I know everything is not clear about backporting.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

async_hooks Issues and PRs related to the async hooks subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants