Skip to content

doc: use HTTPS for links where possible - #6019

Closed
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:good
Closed

doc: use HTTPS for links where possible#6019
Trott wants to merge 1 commit into
nodejs:masterfrom
Trott:good

Conversation

@Trott

@TrottTrott commented Apr 2, 2016

Copy link
Copy Markdown
Member

Pull Request check-list

  • Is the commit message formatted according to [CONTRIBUTING.md][0]?

Affected core subsystem(s)

doc

Description of change

Provide encrypted links to freenode IRC.

Provide encrypted links to freenode IRC.
@TrottTrott added the doc Issues and PRs related to the documentations. label Apr 2, 2016
@jasnell

Copy link
Copy Markdown
Member

LGTM

@stevemao

Copy link
Copy Markdown
Contributor

cc @nodejs/documentation

@Knighton910

Copy link
Copy Markdown

LGTM

2 similar comments
@r-52

r-52 commented Apr 2, 2016

Copy link
Copy Markdown
Contributor

LGTM

@benjamingr

Copy link
Copy Markdown
Member

LGTM

jasnell pushed a commit that referenced this pull request Apr 4, 2016
Provide encrypted links to freenode IRC.
PR-URL: #6019
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
@jasnell

Copy link
Copy Markdown
Member

Landed in 68bd702

@jasnelljasnell closed this Apr 4, 2016
MylesBorins pushed a commit that referenced this pull request Apr 5, 2016
Provide encrypted links to freenode IRC.
PR-URL: #6019
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
This was referenced Apr 5, 2016
MylesBorins pushed a commit that referenced this pull request Apr 11, 2016
Provide encrypted links to freenode IRC.
PR-URL: #6019
Reviewed-By: James M Snell <jasnell@gmail.com>
Reviewed-By: Roman Klauke <romaaan.git@gmail.com>
Reviewed-By: Benjamin Gruenbaum <benjamingr@gmail.com>
@MylesBorinsMylesBorins mentioned this pull request Apr 11, 2016
@Trott
Trott deleted the good branch January 13, 2022 22:42
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

docIssues and PRs related to the documentations.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

7 participants

@Trott@jasnell@stevemao@Knighton910@r-52@benjamingr@MylesBorins