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: fix indent in tls resumption example #3372

Closed
wants to merge 1 commit into from

Conversation

silverwind
Copy link
Contributor

The example added in #3147 was wrongly using a 2-space indent, but markdown requires 4-space for code blocks. This fixes it so this code example is now correctly formatted as a code block.

@silverwind silverwind added the doc Issues and PRs related to the documentations. label Oct 14, 2015
@thefourtheye
Copy link
Contributor

LGTM

@mscdex mscdex added the tls Issues and PRs related to the tls subsystem. label Oct 14, 2015
silverwind added a commit that referenced this pull request Oct 15, 2015
Markdown requires 4-space indent to correctly format code blocks. This
fixes the example so it's correctly presented as code.

PR-URL: #3372
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
@silverwind
Copy link
Contributor Author

Thanks for review, landed in 503f279.

@silverwind silverwind closed this Oct 15, 2015
@silverwind silverwind deleted the doc-fix-markdown branch October 15, 2015 09:32
@MylesBorins
Copy link
Contributor

Should docfixes also land in LTS?

@jasnell
perhaps add lts-watch-v4x

@jasnell
Copy link
Member

jasnell commented Oct 16, 2015

Yes. So long as they're relevant to that branch ;-)

@rvagg rvagg mentioned this pull request Oct 21, 2015
silverwind added a commit that referenced this pull request Oct 26, 2015
Markdown requires 4-space indent to correctly format code blocks. This
fixes the example so it's correctly presented as code.

PR-URL: #3372
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
@jasnell
Copy link
Member

jasnell commented Oct 26, 2015

Landed in v4.x-staging in e223fc7

silverwind added a commit that referenced this pull request Oct 29, 2015
Markdown requires 4-space indent to correctly format code blocks. This
fixes the example so it's correctly presented as code.

PR-URL: #3372
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
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.

5 participants