Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

doc: add missing backticks to doc/api/tls.md #13394

Closed
wants to merge 1 commit into from

Conversation

pbininda
Copy link
Contributor

@pbininda pbininda commented Jun 2, 2017

Fixes: #13382

Checklist
Affected core subsystem(s)

doc, tls

@nodejs-github-bot nodejs-github-bot added doc Issues and PRs related to the documentations. tls Issues and PRs related to the tls subsystem. labels Jun 2, 2017
@refack
Copy link
Contributor

refack commented Jun 2, 2017

@pbininda Thank you very much for the contribution 🥇

I'd recommend you make yourself familiar with our review & landing procedure

Good luck 🤞

@refack refack self-assigned this Jun 2, 2017
@refack
Copy link
Contributor

refack commented Jun 2, 2017

Ohh and welcome 😉
image

@gibfahn
Copy link
Member

gibfahn commented Jun 2, 2017

Yep, the issue is clear in the docs. Thanks for this @pbininda !

image

Copying the dont-land-on labels from #11984

Copy link
Member

@mhdawson mhdawson left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

refack pushed a commit to refack/node that referenced this pull request Jun 4, 2017
PR-URL: nodejs#13394
Fixes: nodejs#13382
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@refack
Copy link
Contributor

refack commented Jun 4, 2017

Landed in 7637347

@refack refack closed this Jun 4, 2017
jasnell pushed a commit that referenced this pull request Jun 5, 2017
PR-URL: #13394
Fixes: #13382
Reviewed-By: Refael Ackermann <refack@gmail.com>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
Reviewed-By: Colin Ihrig <cjihrig@gmail.com>
Reviewed-By: Jeremiah Senkpiel <fishrock123@rocketmail.com>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Reviewed-By: James M Snell <jasnell@gmail.com>
@jasnell jasnell mentioned this pull request Jun 5, 2017
@refack refack removed their assignment Oct 20, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
doc Issues and PRs related to the documentations. tls Issues and PRs related to the tls subsystem.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

doc: incorrect backtick usage
8 participants