X Tutup
Skip to content

tls: replace forEach with for#15053

Closed
mscdex wants to merge 1 commit intonodejs:masterfrom
mscdex:tls-replace-foreach
Closed

tls: replace forEach with for#15053
mscdex wants to merge 1 commit intonodejs:masterfrom
mscdex:tls-replace-foreach

Conversation

@mscdex
Copy link
Copy Markdown
Contributor

@mscdex mscdex commented Aug 27, 2017 •
edited
Loading

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

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines
Affected core subsystem(s)
  • tls

Loading
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@jasnell jasnell jasnell approved these changes

@benjamingr benjamingr benjamingr approved these changes

@lpinca lpinca lpinca approved these changes

@BridgeAR BridgeAR BridgeAR approved these changes

+2 more reviewers

@refack refack refack approved these changes

@daxlab daxlab daxlab approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

tls Issues and PRs related to the tls subsystem.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

8 participants

@mscdex @refack @benjamingr @jasnell @BridgeAR @addaleax @lpinca @daxlab
X Tutup