Skip to content

Conversation

jasnell
Copy link
Member

@jasnell jasnell commented Feb 1, 2018

Oopsie... Forgot to assign the deprecation code when landing #18333

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
Affected core subsystem(s)

crypto, docs

@jasnell jasnell added the fast-track PRs that do not need to wait for 48 hours to land. label Feb 1, 2018
@nodejs-github-bot nodejs-github-bot added the crypto Issues and PRs related to the crypto subsystem. label Feb 1, 2018
@thefourtheye
Copy link
Contributor

Just making an observation. We have DEP0089 after DEP0086.

@BridgeAR BridgeAR added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 1, 2018
@jasnell
Copy link
Member Author

jasnell commented Feb 1, 2018

@jasnell jasnell changed the title crypto: fix unassignd deprecation code doc: fix unassigned deprecation codes Feb 1, 2018
jasnell added a commit that referenced this pull request Feb 1, 2018
Missed when landing the original PRs

PR-URL: #18492
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
@jasnell
Copy link
Member Author

jasnell commented Feb 1, 2018

Landed in 2ac6658

@jasnell jasnell closed this Feb 1, 2018
@addaleax addaleax removed the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Feb 4, 2018
MayaLekova pushed a commit to MayaLekova/node that referenced this pull request May 8, 2018
Missed when landing the original PRs

PR-URL: nodejs#18492
Reviewed-By: Vse Mozhet Byt <[email protected]>
Reviewed-By: Sakthipriyan Vairamani <[email protected]>
Reviewed-By: Tobias Nießen <[email protected]>
Reviewed-By: Ruben Bridgewater <[email protected]>
Reviewed-By: Colin Ihrig <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
crypto Issues and PRs related to the crypto subsystem. fast-track PRs that do not need to wait for 48 hours to land.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

9 participants