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: deeper link to downloads site #23084

Closed
wants to merge 0 commits into from
Closed

Conversation

refack
Copy link
Contributor

@refack refack commented Sep 25, 2018

Use https://nodejs.org/en/download/ instead of https://nodejs.org/ as the link to "downloads".
The /en/ part is a bit unfortunate, but the rest of the doc is in english, so its at least consistent.

Checklist
  • make -j4 test (UNIX), or vcbuild test (Windows) passes
  • documentation is changed or added
  • commit message follows commit guidelines

@refack refack added the doc Issues and PRs related to the documentations. label Sep 25, 2018
@nodejs-github-bot nodejs-github-bot added the doc Issues and PRs related to the documentations. label Sep 25, 2018
@refack
Copy link
Contributor Author

refack commented Sep 25, 2018

/CC @nodejs/documentation @nodejs/website

@mscdex
Copy link
Contributor

mscdex commented Sep 25, 2018

Did you mean s/depper/deeper/ ?

@refack refack changed the title doc: depper link to downloads site doc: deeper link to downloads site Sep 25, 2018
@refack
Copy link
Contributor Author

refack commented Sep 25, 2018

Did you mean s/depper/deeper/ ?

Yes thank you, fixed. (although it could have been "dapper")

@refack refack self-assigned this Sep 25, 2018
@trivikr trivikr added the author ready PRs that have at least one approval, no pending requests for changes, and a CI started. label Sep 26, 2018
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
Copy link
Contributor Author

refack commented Sep 27, 2018

Resume (just for good measure): https://ci.nodejs.org/job/node-test-commit/21824/

@refack refack closed this Sep 28, 2018
refack added a commit to refack/node that referenced this pull request Sep 28, 2018
PR-URL: nodejs#23084
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
@refack refack deleted the readme-dl-link branch September 28, 2018 03:00
@vsemozhetbyt
Copy link
Contributor

Landed in d228c4d

targos pushed a commit that referenced this pull request Sep 28, 2018
PR-URL: #23084
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
targos pushed a commit that referenced this pull request Oct 3, 2018
PR-URL: #23084
Reviewed-By: Anna Henningsen <anna@addaleax.net>
Reviewed-By: Luigi Pinca <luigipinca@gmail.com>
Reviewed-By: Denys Otrishko <shishugi@gmail.com>
Reviewed-By: Trivikram Kamat <trivikr.dev@gmail.com>
Reviewed-By: Sakthipriyan Vairamani <thechargingvolcano@gmail.com>
Reviewed-By: George Adams <george.adams@uk.ibm.com>
Reviewed-By: Michaël Zasso <targos@protonmail.com>
Reviewed-By: Tobias Nießen <tniessen@tnie.de>
Reviewed-By: Michael Dawson <michael_dawson@ca.ibm.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
author ready PRs that have at least one approval, no pending requests for changes, and a CI started. doc Issues and PRs related to the documentations.
Projects
None yet
Development

Successfully merging this pull request may close these issues.