Skip to content

test: replace forEach with for..of in test-http2-server - #49819

Closed
niyashiyas wants to merge 2 commits into
nodejs:mainfrom
niyashiyas:test-2
Closed

test: replace forEach with for..of in test-http2-server#49819
niyashiyas wants to merge 2 commits into
nodejs:mainfrom
niyashiyas:test-2

Conversation

@niyashiyas

Copy link
Copy Markdown
Contributor

Replaced forEach with for..of in test-http2-server-shutdown-options-errors.js

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Sep 23, 2023
@tniessentniessen added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Sep 23, 2023

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Suggested change
};
}

Copy link
Copy Markdown
ContributorAuthor

Choose a reason for hiding this comment

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

Made required changes.

@niyashiyasniyashiyas changed the title test: Replaced forEach with for..of in test-http2-server-shutdown-opt…test: replace forEach with for..of in test-http2-serverSep 30, 2023
@lpincalpinca added commit-queue-squash Add this label to instruct the Commit Queue to squash all the PR commits into the first one. request-ci Add this label to start a Jenkins CI on a PR. labels Oct 14, 2023
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Oct 14, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@lpincalpinca added the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 21, 2023
@nodejs-github-botnodejs-github-bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Oct 21, 2023
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator
Commit Queue failed
- Loading data for nodejs/node/pull/49819
✔ Done loading data for nodejs/node/pull/49819
----------------------------------- PR info ------------------------------------
Title test: replace forEach with for..of in test-http2-server (#49819)
⚠ Could not retrieve the email or name of the PR author's from user's GitHub profile!
Branch niyashiyas:test-2 -> nodejs:main
Labels test, code-and-learn, needs-ci, commit-queue-squash
Commits 2
- test: replace forEach with for..of in test-http2-server
- test: replace forEach with for..of in test-http2-server
Committers 1
- Niya Shiyas PR-URL: https://github.com/nodejs/node/pull/49819
Reviewed-By: Luigi Pinca ------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/49819
Reviewed-By: Luigi Pinca --------------------------------------------------------------------------------
ℹ This PR was created on Sat, 23 Sep 2023 08:24:31 GMT
✔ Approvals: 1
✔ - Luigi Pinca (@lpinca): https://github.com/nodejs/node/pull/49819#pullrequestreview-1678384495
ℹ This PR is being fast-tracked because it is from a Code and Learn event
✘ Last GitHub CI failed
ℹ Last Full PR CI on 2023-10-18T19:45:42Z: https://ci.nodejs.org/job/node-test-pull-request/54971/
- Querying data for job/node-test-pull-request/54971/
✔ Last Jenkins CI successful
--------------------------------------------------------------------------------
✔ Aborted `git node land` session in /home/runner/work/node/node/.ncu
https://github.com/nodejs/node/actions/runs/6595354049

lpinca pushed a commit that referenced this pull request Oct 21, 2023
PR-URL: #49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@lpinca

Copy link
Copy Markdown
Member

Landed in 63e9d02.

@lpincalpinca closed this Oct 21, 2023
targos pushed a commit that referenced this pull request Oct 23, 2023
PR-URL: #49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Oct 23, 2023
targos pushed a commit that referenced this pull request Nov 11, 2023
PR-URL: #49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Nov 12, 2023
targos pushed a commit that referenced this pull request Nov 27, 2023
PR-URL: #49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
@targostargos mentioned this pull request Nov 28, 2023
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
PR-URL: nodejs/node#49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
sercher added a commit to sercher/graaljs that referenced this pull request Apr 25, 2024
PR-URL: nodejs/node#49819
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.commit-queue-failedAn error occurred while landing this pull request using GitHub Actions.commit-queue-squashAdd this label to instruct the Commit Queue to squash all the PR commits into the first one.needs-ciPRs that need a full CI run.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

@niyashiyas@nodejs-github-bot@lpinca@tniessen