Skip to content

crypto: refactor to use more primordials - #36012

Merged
Trott merged 0 commit into
nodejs:masterfrom
aduh95:crypto-primordials
Nov 10, 2020
Merged

crypto: refactor to use more primordials#36012
Trott merged 0 commit into
nodejs:masterfrom
aduh95:crypto-primordials

Conversation

@aduh95

Copy link
Copy Markdown
Contributor
Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • commit message follows commit guidelines

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

Review requested:

  • @nodejs/crypto

@nodejs-github-botnodejs-github-bot added the lib / src Issues and PRs related to general changes in the lib or src directory. label Nov 6, 2020
@aduh95aduh95 added the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2020
@github-actionsgithub-actionsBot removed the request-ci Add this label to start a Jenkins CI on a PR. label Nov 6, 2020
@nodejs-github-bot

This comment has been minimized.

@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@aduh95aduh95 added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Nov 7, 2020
@TrottTrott closed this Nov 10, 2020
@Trott

Copy link
Copy Markdown
Member

Landed in da53a3c

@Trott
Trott merged commit da53a3c into nodejs:masterNov 10, 2020
@aduh95
aduh95 deleted the crypto-primordials branch November 10, 2020 08:38
danielleadams pushed a commit that referenced this pull request Nov 10, 2020
PR-URL: #36012
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Rich Trott <rtrott@gmail.com>
@danielleadamsdanielleadams mentioned this pull request Nov 10, 2020
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.lib / srcIssues and PRs related to general changes in the lib or src directory.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@aduh95@nodejs-github-bot@Trott@jasnell@targos