Skip to content

Conversation

anlexN
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 Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations. labels Aug 26, 2020
@tniessen tniessen added the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2020
@github-actions github-actions bot removed the request-ci Add this label to start a Jenkins CI on a PR. label Sep 2, 2020
@nodejs-github-bot
Copy link
Collaborator

@joyeecheung joyeecheung added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 2, 2020
@rickyes rickyes added the commit-queue Add this label to land a pull request using GitHub Actions. label Sep 10, 2020
@github-actions github-actions bot added commit-queue-failed An error occurred while landing this pull request using GitHub Actions. and removed commit-queue Add this label to land a pull request using GitHub Actions. labels Sep 10, 2020
@github-actions
Copy link
Contributor

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 <[email protected]> (@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 <[email protected]>
------------------------------ Generated metadata ------------------------------
PR-URL: https://github.com/nodejs/node/pull/34930
Reviewed-By: Joyee Cheung 
Reviewed-By: Ricky Zhou <[email protected]>
--------------------------------------------------------------------------------
   ✔  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 [email protected]
Reviewed-By: Ricky Zhou [email protected]

[master 792b08a8] Update crypto.md
Author: anlex N [email protected]
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 pull request Sep 30, 2020
PR-URL: #34930
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Ricky Zhou <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
@gengjiawen
Copy link
Member

Landed in c7e3b0b

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

Thanks for the contribution 😃

danielleadams pushed a commit that referenced this pull request Oct 6, 2020
PR-URL: #34930
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Ricky Zhou <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
@danielleadams danielleadams mentioned this pull request Oct 6, 2020
joesepi pushed a commit to joesepi/node that referenced this pull request Jan 8, 2021
PR-URL: nodejs#34930
Reviewed-By: Joyee Cheung <[email protected]>
Reviewed-By: Ricky Zhou <[email protected]>
Reviewed-By: Trivikram Kamat <[email protected]>
Reviewed-By: Jiawen Geng <[email protected]>
@targos targos removed the commit-queue-failed An error occurred while landing this pull request using GitHub Actions. 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 PRs that have at least one approval, no pending requests for changes, and a CI started. crypto Issues and PRs related to the crypto subsystem. doc Issues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

8 participants