GV

GeezerVison

The World Wide Web, restored to 800 pixels of glory.

Home|About|Gaming cabinet|Archives|How it works|Starter sites

On this site

  • Sign in
  • ActionsCI/CD and automation
  • CodespacesCloud development environments
  • All Features
  • Documentation
  • Training
  • Certifications
  • Build with Copilot
  • Customize Copilot
  • Integrations
  • Trending repos
  • GitHub Sponsors
  • Developer articles
  • Enterprises

This view

  • Original page
  • New address

GeezerVison is showing github.com — build: expand when we run internet tests by mhdawson · Pull Request #49218 · nodejs/node

Conversation

@mhdawson

@mhdawson mhdawson commented Aug 17, 2023

Copy link
Copy Markdown
Member

Refs: #49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did not run because no test under test/internet was changed. Add some of the common paths that include code that might introduce failures in the internet tests.

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

All reactions
@mhdawson
build: expand when we run internet tests
81de6ca
Refs: nodejs#49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson <[email protected]>
@nodejs-github-bot

nodejs-github-bot commented Aug 17, 2023

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/actions
All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@nodejs-github-bot nodejs-github-bot added the meta Issues and PRs related to the general management of the project. label Aug 17, 2023
ruyadorno
ruyadorno approved these changes Aug 17, 2023
View reviewed changes
@mhdawson
squash: make linter happy
b855d5e
anonrig
anonrig approved these changes Aug 17, 2023
View reviewed changes
aduh95
aduh95 approved these changes Aug 18, 2023
View reviewed changes
@aduh95 aduh95 added author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue PRs queued for automated landing through the Commit Queue. labels Aug 18, 2023
MoLow
MoLow approved these changes Aug 19, 2023
View reviewed changes
lpinca
lpinca approved these changes Aug 19, 2023
View reviewed changes
@nodejs-github-bot nodejs-github-bot added commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. and removed commit-queue PRs queued for automated landing through the Commit Queue. labels Aug 19, 2023
@nodejs-github-bot

nodejs-github-bot commented Aug 19, 2023

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/49218
✔  Done loading data for nodejs/node/pull/49218
----------------------------------- PR info ------------------------------------
Title      build: expand when we run internet tests (#49218)
Author     Michael Dawson  (@mhdawson)
Branch     mhdawson:test-internet -> nodejs:main
Labels     meta, author ready
Commits    2
 - build: expand when we run internet tests
 - squash: make linter happy
Committers 2
 - Michael Dawson 
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/49218
Refs: https://github.com/nodejs/node/issues/49203
Reviewed-By: Ruy Adorno 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Moshe Atlow 
Reviewed-By: Luigi Pinca 
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/49218
Refs: https://github.com/nodejs/node/issues/49203
Reviewed-By: Ruy Adorno 
Reviewed-By: Yagiz Nizipli 
Reviewed-By: Antoine du Hamel 
Reviewed-By: Moshe Atlow 
Reviewed-By: Luigi Pinca 
--------------------------------------------------------------------------------
   ℹ  This PR was created on Thu, 17 Aug 2023 20:31:10 GMT
   ✔  Approvals: 5
   ✔  - Ruy Adorno (@ruyadorno) (TSC): https://github.com/nodejs/node/pull/49218#pullrequestreview-1583426778
   ✔  - Yagiz Nizipli (@anonrig) (TSC): https://github.com/nodejs/node/pull/49218#pullrequestreview-1583478858
   ✔  - Antoine du Hamel (@aduh95) (TSC): https://github.com/nodejs/node/pull/49218#pullrequestreview-1584225277
   ✔  - Moshe Atlow (@MoLow) (TSC): https://github.com/nodejs/node/pull/49218#pullrequestreview-1585710027
   ✔  - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/49218#pullrequestreview-1585716605
   ✔  Last GitHub CI successful
   ℹ  Green GitHub CI is sufficient
--------------------------------------------------------------------------------
   ✔  No git cherry-pick in progress
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/main up to date...
From https://github.com/nodejs/node
 * branch                  main       -> FETCH_HEAD
✔  origin/main is now up-to-date
- Downloading patch for 49218
From https://github.com/nodejs/node
 * branch                  refs/pull/49218/merge -> FETCH_HEAD
✔  Fetched commits as 41a3a1daa28b..b855d5e9d24e
--------------------------------------------------------------------------------
[main f899b45376] build: expand when we run internet tests
 Author: Michael Dawson 
 Date: Thu Aug 17 16:17:22 2023 -0400
 1 file changed, 10 insertions(+), 2 deletions(-)
[main 9eb94b9abe] squash: make linter happy
 Author: Michael Dawson 
 Date: Thu Aug 17 16:43:37 2023 -0400
 1 file changed, 8 insertions(+), 8 deletions(-)
   ✔  Patches applied
There are 2 commits in the PR. Attempting autorebase.
Rebasing (2/4)

Executing: git node land --amend --yes
⚠ Found Refs: #49203, skipping..
--------------------------------- New Message ----------------------------------
build: expand when we run internet tests

Refs: #49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson [email protected]
PR-URL: #49218
Reviewed-By: Ruy Adorno [email protected]
Reviewed-By: Yagiz Nizipli [email protected]
Reviewed-By: Antoine du Hamel [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Luigi Pinca [email protected]

[detached HEAD 49e8ddcc49] build: expand when we run internet tests
Author: Michael Dawson [email protected]
Date: Thu Aug 17 16:17:22 2023 -0400
1 file changed, 10 insertions(+), 2 deletions(-)
Rebasing (3/4)
Rebasing (4/4)

Executing: git node land --amend --yes
--------------------------------- New Message ----------------------------------
squash: make linter happy

PR-URL: #49218
Refs: #49203
Reviewed-By: Ruy Adorno [email protected]
Reviewed-By: Yagiz Nizipli [email protected]
Reviewed-By: Antoine du Hamel [email protected]
Reviewed-By: Moshe Atlow [email protected]
Reviewed-By: Luigi Pinca [email protected]

[detached HEAD ea0d13c471] squash: make linter happy
Author: Michael Dawson [email protected]
Date: Thu Aug 17 16:43:37 2023 -0400
1 file changed, 8 insertions(+), 8 deletions(-)

Successfully rebased and updated refs/heads/main.

ℹ Add commit-queue-squash label to land the PR as one commit, or commit-queue-rebase to land as separate commits.

https://github.com/nodejs/node/actions/runs/5913403184

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

@aduh95 aduh95 added commit-queue PRs queued for automated landing through the Commit Queue. commit-queue-squash PRs the Commit Queue should land as one squashed commit. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Aug 19, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Aug 19, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 5026293 into nodejs:main Aug 19, 2023
@nodejs-github-bot

nodejs-github-bot commented Aug 19, 2023

Copy link
Copy Markdown
Collaborator

Landed in 5026293

All reactions

Sorry, something went wrong.

Uh oh!

There was an error while loading. Please reload this page.

UlisesGascon pushed a commit that referenced this pull request Sep 10, 2023
@mhdawson @UlisesGascon
build: expand when we run internet tests
34a2590
Refs: #49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #49218
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@UlisesGascon UlisesGascon mentioned this pull request Sep 10, 2023
v20.7.0 proposal #49592
Merged
targos pushed a commit that referenced this pull request Nov 27, 2023
@mhdawson @targos
build: expand when we run internet tests
9e1c531
Refs: #49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: #49218
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
@targos targos mentioned this pull request Nov 28, 2023
v18.19.0 release proposal #50953
Merged
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
build: expand when we run internet tests
aee579f
Refs: nodejs/node#49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs/node#49218
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
@sercher
build: expand when we run internet tests
36dbbc7
Refs: nodejs/node#49203

Changes slipped into v18.x regressed
test/internet/test-dns-ipv6 as I assume the action did
not run because no test under test/internet was changed.
Add some of the common paths that include code that might
introduce failures in the internet tests.

Signed-off-by: Michael Dawson <[email protected]>
PR-URL: nodejs/node#49218
Reviewed-By: Ruy Adorno <[email protected]>
Reviewed-By: Yagiz Nizipli <[email protected]>
Reviewed-By: Antoine du Hamel <[email protected]>
Reviewed-By: Moshe Atlow <[email protected]>
Reviewed-By: Luigi Pinca <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@ruyadorno ruyadorno ruyadorno approved these changes
@lpinca lpinca lpinca approved these changes
@anonrig anonrig anonrig approved these changes
@MoLow MoLow MoLow approved these changes
@aduh95 aduh95 aduh95 approved these changes

Uh oh!

There was an error while loading. Please reload this page.

Assignees

No one assigned

Labels

author ready PRs with CI started, the required approvals, and no outstanding review comments. commit-queue-squash PRs the Commit Queue should land as one squashed commit. meta Issues and PRs related to the general management of the project.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Uh oh!

There was an error while loading. Please reload this page.

7 participants

@mhdawson @nodejs-github-bot @ruyadorno @lpinca @anonrig @MoLow @aduh95
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
You are visitor 01402503 — GeezerVison does not store the pages you look up.
Best viewed with Netscape Navigator 4.7 or Internet Explorer 6 at 800×600. No cookies required.