GeezerVisonThe World Wide Web, restored to 800 pixels of glory. | |||||
| Home|About|Gaming cabinet|Archives|How it works|Starter sites | |||||
GeezerVison is showing github.com — test: replace foreach with for in test-https-simple.js by Shikha2410 · Pull Request #49793 · nodejs/node Conversation
|
|
Thank you for approving @tniessen. Anything I can do to help resolve the build failures in CI? |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
I had triggered a re-run for the failed actions. |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Commit Queue failed- Loading data for nodejs/node/pull/49793 ✔ Done loading data for nodejs/node/pull/49793 ----------------------------------- PR info ------------------------------------ Title test: replace foreach with for in test-https-simple.js (#49793) Author Shikha Mehta (@Shikha2410, first-time contributor) Branch Shikha2410:replace-foreach -> nodejs:main Labels test, code-and-learn, author ready, needs-ci Commits 1 - test: replace foreach with for in test-https-simple.js Committers 1 - Shikha Mehta PR-URL: https://github.com/nodejs/node/pull/49793 Fixes: https://docs.google.com/spreadsheets/d/1kjaxxXX2zGgJUoqj-7U3EYf_0dvSuI-N6YPan4Hye_4/edit#gid=0&range=100:100 Reviewed-By: Tobias Nießen Reviewed-By: Jithil P Ponnan ------------------------------ Generated metadata ------------------------------ PR-URL: https://github.com/nodejs/node/pull/49793 Fixes: https://docs.google.com/spreadsheets/d/1kjaxxXX2zGgJUoqj-7U3EYf_0dvSuI-N6YPan4Hye_4/edit#gid=0&range=100:100 Reviewed-By: Tobias Nießen Reviewed-By: Jithil P Ponnan -------------------------------------------------------------------------------- ℹ This PR was created on Fri, 22 Sep 2023 22:40:08 GMT ✔ Approvals: 2 ✔ - Tobias Nießen (@tniessen) (TSC): https://github.com/nodejs/node/pull/49793#pullrequestreview-1739958150 ✔ - Jithil P Ponnan (@MrJithil): https://github.com/nodejs/node/pull/49793#pullrequestreview-1741332180 ℹ This PR is being fast-tracked because it is from a Code and Learn event ✔ Last GitHub CI successful ℹ Last Full PR CI on 2023-11-21T13:00:20Z: https://ci.nodejs.org/job/node-test-pull-request/55823/ - Querying data for job/node-test-pull-request/55823/ ✔ Last Jenkins CI successful -------------------------------------------------------------------------------- ✔ 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 49793 From https://github.com/nodejs/node * branch refs/pull/49793/merge -> FETCH_HEAD ✔ Fetched commits as 1858341377c2..cc7938d71ac5 -------------------------------------------------------------------------------- [main 95eeeaaf91] test: replace foreach with for in test-https-simple.js Author: Shikha Mehta Date: Fri Sep 22 15:22:18 2023 -0700 1 file changed, 2 insertions(+), 2 deletions(-) ✔ Patches applied -------------------------------------------------------------------------------- --------------------------------- New Message ---------------------------------- test: replace foreach with for in test-https-simple.jshttps://github.com/nodejs/node/actions/runs/6950894323 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
|
Landed in 4e713a3 |
Sorry, something went wrong.
There was an error while loading. Please reload this page.
Fixes: nodejs#50818 PR-URL: nodejs#49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
Fixes: #50818 PR-URL: #49793 Reviewed-By: Tobias Nießen <[email protected]> Reviewed-By: Jithil P Ponnan <[email protected]>
There was an error while loading. Please reload this page.
Successfully merging this pull request may close these issues.
test: replace foreach with for in test-https-simple.js
There was an error while loading. Please reload this page.