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 — test: replace foreach with for in test-https-simple.js by Shikha2410 · Pull Request #49793 · nodejs/node

Conversation

@Shikha2410

@Shikha2410 Shikha2410 commented Sep 22, 2023 •
edited by tniessen
Loading

Uh oh!

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

Copy link
Copy Markdown

Fixes: #50818

Sorry, something went wrong.

Uh oh!

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

All reactions
@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure. labels Sep 22, 2023
tniessen
tniessen reviewed Sep 22, 2023
View reviewed changes

@tniessen tniessen left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please amend the commit message and force-push as described here. The prefix should be test:, not core:.

Sorry, something went wrong.

Uh oh!

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

Shikha2410 reacted with thumbs up emoji
All reactions
  • 👍 1 reaction
Comment thread test/parallel/test-https-simple.js Resolved

Uh oh!

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

@tniessen tniessen added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Sep 22, 2023
@Shikha2410
Shikha2410 force-pushed the replace-foreach branch from 117094b to b484a3f Compare September 22, 2023 23:42
@Shikha2410 Shikha2410 changed the title core: replace foreach with for in test-https-simple.js test: replace foreach with for in test-https-simple.js Sep 22, 2023
tniessen
tniessen approved these changes Sep 23, 2023
View reviewed changes
Shikha Mehta
test: replace foreach with for in test-https-simple.js
cc7938d
Fixes: nodejs#50818
@Shikha2410
Shikha2410 force-pushed the replace-foreach branch from b484a3f to cc7938d Compare November 20, 2023 07:39
@tniessen tniessen added the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Nov 20, 2023
tniessen
tniessen approved these changes Nov 20, 2023
View reviewed changes

@tniessen tniessen left a comment

Copy link
Copy Markdown
Member

There was a problem hiding this comment.

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you @Shikha2410! And apologies about the delay.

Sorry, something went wrong.

Uh oh!

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

Shikha2410 reacted with thumbs up emoji Shikha2410 reacted with rocket emoji
All reactions
  • 👍 1 reaction
  • 🚀 1 reaction
@tniessen tniessen added the author ready PRs with CI started, the required approvals, and no outstanding review comments. label Nov 20, 2023
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. Only starts once the PR has an approving review. label Nov 20, 2023
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
@Shikha2410

Shikha2410 commented Nov 21, 2023

Copy link
Copy Markdown
Author

Thank you for approving @tniessen. Anything I can do to help resolve the build failures in CI?

All reactions

Sorry, something went wrong.

Uh oh!

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

@MrJithil

MrJithil commented Nov 21, 2023

Copy link
Copy Markdown
Member

I had triggered a re-run for the failed actions.

Shikha2410 reacted with thumbs up emoji
All reactions
  • 👍 1 reaction

Sorry, something went wrong.

Uh oh!

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

@nodejs-github-bot

This comment was marked as outdated.

Sign in to view
MrJithil
MrJithil approved these changes Nov 21, 2023
View reviewed changes
@nodejs-github-bot

nodejs-github-bot commented Nov 21, 2023

Copy link
Copy Markdown
Collaborator

CI: https://ci.nodejs.org/job/node-test-pull-request/55823/

All reactions

Sorry, something went wrong.

Uh oh!

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

@tniessen tniessen added the commit-queue PRs queued for automated landing through the Commit Queue. label Nov 21, 2023
@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 Nov 21, 2023
@nodejs-github-bot

nodejs-github-bot commented Nov 21, 2023

Copy link
Copy Markdown
Collaborator
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.js

Fixes: #50818
PR-URL: #49793
Fixes: https://docs.google.com/spreadsheets/d/1kjaxxXX2zGgJUoqj-7U3EYf_0dvSuI-N6YPan4Hye_4/edit#gid=0&range=100:100
Reviewed-By: Tobias Nießen [email protected]
Reviewed-By: Jithil P Ponnan [email protected]

[main 23d682ade1] test: replace foreach with for in test-https-simple.js
Author: Shikha Mehta [email protected]
Date: Fri Sep 22 15:22:18 2023 -0700
1 file changed, 2 insertions(+), 2 deletions(-)
✖ 23d682ade1e7b5fc3f4744013eb8c2e971ed2053
✔ 0:0 no Co-authored-by metadata co-authored-by-is-trailer
✔ 1:7 Valid fixes URL. fixes-url
✖ 3:7 Fixes must be a GitHub URL. fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 2:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✔ 0:0 valid subsystems subsystem
✔ 0:0 Title is formatted correctly. title-format
⚠ 0:50 Title should be <= 50 columns. title-length

ℹ Please fix the commit message and try again.
Please manually ammend the commit message, by running
git commit --amend
Once commit message is fixed, finish the landing command running
git node land --continue

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

All reactions

Sorry, something went wrong.

Uh oh!

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

@tniessen tniessen added commit-queue PRs queued for automated landing through the Commit Queue. and removed commit-queue-failed PRs whose Commit Queue landing failed and need manual intervention before retrying. labels Nov 23, 2023
@nodejs-github-bot nodejs-github-bot removed the commit-queue PRs queued for automated landing through the Commit Queue. label Nov 23, 2023
@nodejs-github-bot
nodejs-github-bot merged commit 4e713a3 into nodejs:main Nov 23, 2023
@nodejs-github-bot

nodejs-github-bot commented Nov 23, 2023

Copy link
Copy Markdown
Collaborator

Landed in 4e713a3

All reactions

Sorry, something went wrong.

Uh oh!

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

lucshi pushed a commit to lucshi/node that referenced this pull request Nov 27, 2023
Shikha Mehta Lei Shi
test: replace foreach with for in test-https-simple.js
4a3ce61
Fixes: nodejs#50818
PR-URL: nodejs#49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
RafaelGSS pushed a commit that referenced this pull request Nov 27, 2023
Shikha Mehta @RafaelGSS
test: replace foreach with for in test-https-simple.js
b851895
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
@RafaelGSS RafaelGSS mentioned this pull request Nov 28, 2023
v21.3.0 proposal #50954
Merged
RafaelGSS pushed a commit that referenced this pull request Nov 29, 2023
Shikha Mehta @RafaelGSS
test: replace foreach with for in test-https-simple.js
9d76de1
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
Shikha Mehta @UlisesGascon
test: replace foreach with for in test-https-simple.js
6a64824
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
@UlisesGascon UlisesGascon mentioned this pull request Dec 12, 2023
v20.11.0 proposal #51124
Merged
UlisesGascon pushed a commit that referenced this pull request Dec 13, 2023
Shikha Mehta @UlisesGascon
test: replace foreach with for in test-https-simple.js
622cf68
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 15, 2023
Shikha Mehta @UlisesGascon
test: replace foreach with for in test-https-simple.js
347e92a
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 19, 2023
Shikha Mehta @UlisesGascon
test: replace foreach with for in test-https-simple.js
107b5e6
Fixes: #50818
PR-URL: #49793
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Jithil P Ponnan <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@tniessen tniessen tniessen approved these changes
@MrJithil MrJithil MrJithil 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. code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. needs-ci PRs that need a full CI run. test Issues and PRs related to Node.js core tests and test infrastructure.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

test: replace foreach with for in test-https-simple.js

Uh oh!

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

4 participants

@Shikha2410 @nodejs-github-bot @MrJithil @tniessen
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.