Skip to content

test: convert test-async-local-storage forEach to for..of loop - #65270

Open
freida-code wants to merge 1 commit into
nodejs:mainfrom
freida-code:convert-test-async-local-storage-bind-loop
Open

test: convert test-async-local-storage forEach to for..of loop#65270
freida-code wants to merge 1 commit into
nodejs:mainfrom
freida-code:convert-test-async-local-storage-bind-loop

Conversation

@freida-code

@freida-code freida-code commented Aug 13, 2026

Copy link
Copy Markdown

Update async-local-storage-bind test to use for..of for loop instead of .forEach() to avoid allocating a callback per iteration and have a cleaner stack.

Completed during the Node JS workshop at RenderATL

@nodejs-github-bot nodejs-github-bot added needs-ci PRs that need a full CI run. test Issues and PRs related to the tests. labels Aug 13, 2026
@freida-code

Copy link
Copy Markdown
Author

@jasnell Tagging you for review! I look forward to future contributions! Thank you!

@jasnell jasnell added the code-and-learn Issues related to the Code-and-Learn events and PRs submitted during the events. label Aug 13, 2026
@jasnell jasnell added the request-ci Add this label to start a Jenkins CI on a PR. label Aug 13, 2026
@github-actions github-actions Bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Aug 13, 2026
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@freida-code freida-code changed the title test: convert test-async-local-storage-bind to async loop test: convert test-async-local-storage forEach to for..of loop Aug 13, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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 the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants