Skip to content

Conversation

@jabali2004
Copy link
Contributor

@jabali2004jabali2004 commented Nov 7, 2023

Replaced forEach with for of in test/parralel/test-crypto-getcipherinfo.js
#NodeConfEU

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Nov 7, 2023
@marco-ippolitomarco-ippolito added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Nov 7, 2023
@jabali2004jabali2004force-pushed the chore/convert-forEach-to-forOf branch from 1a345e3 to b990ef5CompareNovember 7, 2023 15:21
@lpincalpinca added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@github-actionsgithub-actionsbot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 7, 2023
@nodejs-github-bot
Copy link
Collaborator

@nodejs-github-bot
Copy link
Collaborator

@tniessentniessen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 8, 2023
@debadree25debadree25 added the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 10, 2023
@nodejs-github-botnodejs-github-bot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Nov 10, 2023
@nodejs-github-botnodejs-github-bot merged commit 38cd4f7 into nodejs:mainNov 10, 2023
@nodejs-github-bot
Copy link
Collaborator

Landed in 38cd4f7

@jabali2004jabali2004 deleted the chore/convert-forEach-to-forOf branch November 10, 2023 14:15
targos pushed a commit that referenced this pull request Nov 11, 2023
PR-URL: #50597 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@targostargos mentioned this pull request Nov 12, 2023
targos pushed a commit that referenced this pull request Nov 14, 2023
PR-URL: #50597 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
UlisesGascon pushed a commit that referenced this pull request Dec 11, 2023
PR-URL: #50597 Reviewed-By: Marco Ippolito <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@UlisesGasconUlisesGascon mentioned this pull request Dec 12, 2023
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

author readyPRs that have at least one approval, no pending requests for changes, and a CI started.code-and-learnIssues related to the Code-and-Learn events and PRs submitted during the events.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.

6 participants

@jabali2004@nodejs-github-bot@lpinca@tniessen@marco-ippolito@debadree25