Skip to content

Conversation

@RaisinTen
Copy link
Member

Signed-off-by: Darshan Sen [email protected]

@nodejs-github-botnodejs-github-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 Oct 14, 2021
@nodejs-github-bot

This comment has been minimized.

@RaisinTenRaisinTenforce-pushed the src,crypto/use-std-variant-in-DH-params branch from 869f42a to 29a2559CompareOctober 14, 2021 14:11
@nodejs-github-bot

This comment has been minimized.

@richardlau
Copy link
Member

Added dont-land- labels as std::variant is a C++17 feature.

@RaisinTenRaisinTen added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Oct 14, 2021
@nodejs-github-bot

This comment has been minimized.

@RaisinTenRaisinTen removed the needs-ci PRs that need a full CI run. label Oct 15, 2021
@nodejs-github-bot
Copy link
Collaborator

@github-actionsgithub-actionsbot removed the commit-queue Add this label to land a pull request using GitHub Actions. label Oct 22, 2021
@github-actions
Copy link
Contributor

Landed in 68dc180...5c7af3f

nodejs-github-bot pushed a commit that referenced this pull request Oct 22, 2021
Signed-off-by: Darshan Sen <[email protected]> PR-URL: #40457 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@RaisinTenRaisinTen deleted the src,crypto/use-std-variant-in-DH-params branch October 22, 2021 14:45
targos pushed a commit that referenced this pull request Oct 23, 2021
Signed-off-by: Darshan Sen <[email protected]> PR-URL: #40457 Reviewed-By: Anna Henningsen <[email protected]> Reviewed-By: Richard Lau <[email protected]> Reviewed-By: Tobias Nießen <[email protected]>
@targostargos mentioned this pull request Nov 8, 2021
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.c++Issues and PRs that require attention from people who are familiar with C++.cryptoIssues and PRs related to the crypto subsystem.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants

@RaisinTen@nodejs-github-bot@richardlau@addaleax@tniessen