Skip to content

test: remove unused ecdhPeerKey - #36942

Closed
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:webscrypto_test_wrap_unwrap
Closed

test: remove unused ecdhPeerKey#36942
danbev wants to merge 1 commit into
nodejs:masterfrom
danbev:webscrypto_test_wrap_unwrap

Conversation

@danbev

Copy link
Copy Markdown
Contributor

This commit removed ecdhPeerKey from test-webcrypto-wrap-unwrap.js which
seems to be unsued.

This commit removed ecdhPeerKey from test-webcrypto-wrap-unwrap.js which
seems to be unsued.
@nodejs-github-botnodejs-github-bot added the test Issues and PRs related to the tests. label Jan 15, 2021
@nodejs-github-bot

Copy link
Copy Markdown
Collaborator

@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 Jan 18, 2021
@jasnell

Copy link
Copy Markdown
Member

Landed in c880965

@jasnelljasnell closed this Jan 18, 2021
jasnell pushed a commit that referenced this pull request Jan 18, 2021
This commit removed ecdhPeerKey from test-webcrypto-wrap-unwrap.js which
seems to be unsued.
PR-URL: #36942
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
ruyadorno pushed a commit that referenced this pull request Jan 22, 2021
This commit removed ecdhPeerKey from test-webcrypto-wrap-unwrap.js which
seems to be unsued.
PR-URL: #36942
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
@ruyadornoruyadorno mentioned this pull request Jan 22, 2021
ruyadorno pushed a commit that referenced this pull request Jan 25, 2021
This commit removed ecdhPeerKey from test-webcrypto-wrap-unwrap.js which
seems to be unsued.
PR-URL: #36942
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
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.testIssues and PRs related to the tests.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants

@danbev@nodejs-github-bot@jasnell@cjihrig@targos@aduh95