Skip to content

Conversation

@arkerone
Copy link
Contributor

I found a typo : "byteLenth" -> "byteLength"

@nodejs-github-botnodejs-github-bot added needs-ci PRs that need a full CI run. tls Issues and PRs related to the tls subsystem. labels Apr 7, 2021
@aduh95aduh95 added author ready PRs that have at least one approval, no pending requests for changes, and a CI started. fast-track PRs that do not need to wait for 48 hours to land. and removed needs-ci PRs that need a full CI run. labels Apr 7, 2021
@aduh95
Copy link
Contributor

There's no reason to make this one wait the 48 hours to land. Collaborators, please 👍🏻 to fast-track

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot
Copy link
Collaborator

`byteLenth` -> `byteLength`. PR-URL: nodejs#38129 Reviewed-By: Antoine du Hamel <[email protected]> Reviewed-By: Luigi Pinca <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Darshan Sen <[email protected]> Reviewed-By: James M Snell <[email protected]> Reviewed-By: Colin Ihrig <[email protected]>
@aduh95aduh95force-pushed the fix/typo-tls-common branch from 000048f to b16e79eCompareApril 8, 2021 08:36
@aduh95
Copy link
Contributor

Landed in b16e79e

@aduh95aduh95 merged commit b16e79e into nodejs:masterApr 8, 2021
@arkeronearkerone deleted the fix/typo-tls-common branch April 8, 2021 12:22
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.fast-trackPRs that do not need to wait for 48 hours to land.tlsIssues and PRs related to the tls subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

10 participants

@arkerone@aduh95@nodejs-github-bot@jasnell@lpinca@cjihrig@richardlau@RaisinTen@VoltrexKeyva@targos