FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: use sentence case in headers in src/crypto/README.md by Trott · Pull Request #38524 · nodejs/node · GitHub

/ node Public

doc: use sentence case in headers in src/crypto/README.md - #38524

Merged
Trott merged 1 commit into
nodejs:masterfrom
Trott:crypto-readme
May 4, 2021
Merged

doc: use sentence case in headers in src/crypto/README.md#38524
Trott merged 1 commit into
nodejs:masterfrom
Trott:crypto-readme

Conversation

Trott commented May 3, 2021

Copy link
Copy Markdown
Member

Use sentence case consistently in headers in src/crypto/README.md.

github-actions Bot added c++ Issues and PRs that require attention from people who are familiar with C++. crypto Issues and PRs related to the crypto subsystem. needs-ci PRs that need a full CI run. labels May 3, 2021
panva added the fast-track PRs that do not need to wait for 48 hours to land. label May 4, 2021

github-actions Bot commented May 4, 2021

Copy link
Copy Markdown
Contributor

Fast-track has been requested by @panva. Please 👍 to approve.

panva added doc Issues and PRs related to the documentations. and removed c++ Issues and PRs that require attention from people who are familiar with C++. needs-ci PRs that need a full CI run. labels May 4, 2021
Use sentence case consistently in headers in src/crypto/README.md.

PR-URL: nodejs#38524
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
Trott force-pushed the crypto-readme branch from 22c94d9 to f71166b Compare May 4, 2021 13:33

Trott commented May 4, 2021

Copy link
Copy Markdown
Member Author

Landed in f71166b

Trott merged commit f71166b into nodejs:master May 4, 2021
Trott deleted the crypto-readme branch May 4, 2021 13:34
targos pushed a commit that referenced this pull request May 17, 2021
Use sentence case consistently in headers in src/crypto/README.md.

PR-URL: #38524
Reviewed-By: Antoine du Hamel <duhamelantoine1995@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Zijian Liu <lxxyxzj@gmail.com>
Reviewed-By: Gireesh Punathil <gpunathi@in.ibm.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. fast-track PRs that do not need to wait for 48 hours to land.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants


Back | FazBrowse Home | New Git URL