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

test: add test case for reverted 17.7 regression #42283

Merged
merged 1 commit into from Mar 11, 2022

Conversation

Copy link
Member

@Trott Trott commented Mar 10, 2022

Refs: #42279

The first commit in this PR is #42280. Once that lands, it can be removed from here. Marking as blocked for now.

@Trott Trott added the blocked label Mar 10, 2022
@nodejs-github-bot nodejs-github-bot added needs-ci url labels Mar 10, 2022
@Trott Trott removed the blocked label Mar 10, 2022
@Trott
Copy link
Author

@Trott Trott commented Mar 10, 2022

Unblocked.

@Trott Trott added the request-ci label Mar 10, 2022
@Trott Trott added the author ready label Mar 10, 2022
@github-actions github-actions bot removed the request-ci label Mar 10, 2022
@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Mar 10, 2022

@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Mar 10, 2022

@Trott Trott added the fast-track label Mar 10, 2022
@github-actions
Copy link

@github-actions github-actions bot commented Mar 10, 2022

Fast-track has been requested by @Trott. Please 👍 to approve.

@Trott Trott added the commit-queue label Mar 11, 2022
@nodejs-github-bot nodejs-github-bot removed the commit-queue label Mar 11, 2022
@nodejs-github-bot nodejs-github-bot merged commit bf357b7 into nodejs:master Mar 11, 2022
59 checks passed
@nodejs-github-bot
Copy link

@nodejs-github-bot nodejs-github-bot commented Mar 11, 2022

Landed in bf357b7

@Trott Trott deleted the url-relative branch Mar 11, 2022
bengl pushed a commit that referenced this issue Mar 21, 2022
Refs: #42279

PR-URL: #42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@bengl bengl mentioned this pull request Mar 21, 2022
danielleadams pushed a commit to danielleadams/node that referenced this issue Apr 21, 2022
Refs: nodejs#42279

PR-URL: nodejs#42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Apr 24, 2022
Refs: #42279

PR-URL: #42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Apr 24, 2022
Refs: #42279

PR-URL: #42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
danielleadams pushed a commit that referenced this issue Apr 24, 2022
Refs: #42279

PR-URL: #42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
xtx1130 pushed a commit to xtx1130/node that referenced this issue Apr 25, 2022
Refs: nodejs#42279

PR-URL: nodejs#42283
Reviewed-By: Richard Lau <rlau@redhat.com>
Reviewed-By: Darshan Sen <raisinten@gmail.com>
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Mestery <mestery@protonmail.com>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready fast-track needs-ci url
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

10 participants