Sitelet https://web.archive.org/web/20220423174247/https://github.com/nodejs/node/pull/34930
Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Update crypto.md #34930

Closed
wants to merge 1 commit into from
Closed

Update crypto.md #34930

wants to merge 1 commit into from

Conversation

Copy link
Contributor

@anlexN anlexN commented Aug 26, 2020

align to function signature

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines

align to function signature
@nodejs-github-bot nodejs-github-bot added crypto doc labels Aug 26, 2020
@tniessen tniessen added the request-ci label Sep 2, 2020
@github-actions github-actions bot removed the request-ci label Sep 2, 2020
@nodejs-github-bot
Copy link
Contributor

@nodejs-github-bot nodejs-github-bot commented Sep 2, 2020

@joyeecheung joyeecheung added the author ready label Sep 2, 2020
@rickyes rickyes added the commit-queue label Sep 10, 2020
@github-actions github-actions bot added commit-queue-failed and removed commit-queue labels Sep 10, 2020
@github-actions
Copy link

@github-actions github-actions bot commented Sep 10, 2020

Commit Queue failed
- Loading data for nodejs/node/pull/34930
✔  Done loading data for nodejs/node/pull/34930
----------------------------------- PR info ------------------------------------
Title      Update crypto.md (#34930)
Author     anlex N <1293006794@qq.com> (@anlexN, first-time contributor)
Branch     anlexN:patch-1 -> nodejs:master
Labels     author ready, crypto, doc
Commits    1
 - Update crypto.md
Committers 1
 - GitHub 
PR-URL: https://github.com/nodejs/node/pull/34930
Reviewed-By: Joyee Cheung 
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/34930
Reviewed-By: Joyee Cheung 
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
--------------------------------------------------------------------------------
   ✔  Last GitHub Actions successful
   ℹ  Doc-only changes
   ℹ  This PR was created on Wed, 26 Aug 2020 09:29:44 GMT
   ✔  Approvals: 2
   ✔  - Joyee Cheung (@joyeecheung) (TSC): https://github.com/nodejs/node/pull/34930#pullrequestreview-476765948
   ✔  - Ricky Zhou (@rickyes): https://github.com/nodejs/node/pull/34930#pullrequestreview-486026424
--------------------------------------------------------------------------------
   ✔  No git am in progress
   ✔  No git rebase in progress
--------------------------------------------------------------------------------
- Bringing origin/master up to date...
From https://github.com/nodejs/node
 * branch              master     -> FETCH_HEAD
✔  origin/master is now up-to-date
- Downloading patch for 34930
✔  Downloaded patch to /home/runner/work/node/node/.ncu/34930/patch
--------------------------------------------------------------------------------
Applying: Update crypto.md
   ✔  Patches applied
--------------------------------------------------------------------------------
--------------------------------- New Message ----------------------------------
Update crypto.md

align to function signature

PR-URL: #34930
Reviewed-By: Joyee Cheung joyeec9h3@gmail.com
Reviewed-By: Ricky Zhou 0x19951125@gmail.com

[master 792b08a8] Update crypto.md
Author: anlex N 1293006794@qq.com
Date: Wed Aug 26 17:26:54 2020 +0800
1 file changed, 4 insertions(+), 4 deletions(-)
✖ 792b08a8687906710f2fb57b214109e0c93d02b6
✔ 0:0 skipping fixes-url fixes-url
✔ 0:0 blank line after title line-after-title
✔ 0:0 line-lengths are valid line-length
✔ 0:0 metadata is at end of message metadata-end
✔ 3:8 PR-URL is valid. pr-url
✔ 0:0 reviewers are valid reviewers
✖ 0:0 Missing subsystem. subsystem
✔ 0:0 Title is formatted correctly. title-format
✔ 0:0 Title is <= 50 columns. title-length

ℹ Please fix the commit message and try again.

gengjiawen pushed a commit that referenced this issue Sep 30, 2020
PR-URL: #34930
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@gengjiawen
Copy link
Member

@gengjiawen gengjiawen commented Sep 30, 2020

Landed in c7e3b0b

@gengjiawen gengjiawen closed this Sep 30, 2020
@gengjiawen
Copy link
Member

@gengjiawen gengjiawen commented Sep 30, 2020

Thanks for the contribution 😃

danielleadams pushed a commit that referenced this issue Oct 6, 2020
PR-URL: #34930
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@danielleadams danielleadams mentioned this pull request Oct 6, 2020
joesepi pushed a commit to joesepi/node that referenced this issue Jan 8, 2021
PR-URL: nodejs#34930
Reviewed-By: Joyee Cheung <joyeec9h3@gmail.com>
Reviewed-By: Ricky Zhou <0x19951125@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Jiawen Geng <technicalcute@gmail.com>
@targos targos removed the commit-queue-failed label Sep 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready crypto doc
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

8 participants