FazBrowse GitHub Viewer | Trending |
URL:
| Home
Tools: [Download Repo ZIP]   [Original HTTPS Page]

doc: jenkins-iojs.nodesource.com -> ci.nodejs.org, iojs.org/download -> nodejs.org/download by mgol · Pull Request #2886 · nodejs/node · GitHub

/ node Public

doc: jenkins-iojs.nodesource.com -> ci.nodejs.org, iojs.org/download -> nodejs.org/download - #2886

Closed
mgol wants to merge 1 commit into
nodejs:masterfrom
mgol:io2node
ConversationCommits (1)ChecksFiles changed
Closed

doc: jenkins-iojs.nodesource.com -> ci.nodejs.org, iojs.org/download -> nodejs.org/download#2886
mgol wants to merge 1 commit into
nodejs:masterfrom
mgol:io2node

Conversation

mgol commented Sep 15, 2015

Copy link
Copy Markdown
Contributor

The Jenkins URL has changed from https://jenkins-iojs.nodesource.com
to https://ci.nodejs.org. The former address now redirects to the
latter.

The new download location is https://nodejs.org/download/release/;
the configure script was still referring to
https://iojs.org/download/release/.

(there are separate commits for these changes)

mscdex added the doc Issues and PRs related to the documentations. label Sep 15, 2015

rvagg commented Sep 16, 2015

Copy link
Copy Markdown
Member

lgtm

evanlucas commented Sep 16, 2015

Copy link
Copy Markdown
Contributor

LGTM

thefourtheye reviewed Sep 16, 2015
View reviewed changes
Comment thread doc/releases.md Outdated

thefourtheye Sep 16, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

I don't think we use this job anymore. Probably you can fix this as well.

Fishrock123 Sep 16, 2015

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Should be https://ci.nodejs.org/job/node-test-pull-request/

mgol Sep 16, 2015

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Choose a reason Spam Abuse Off Topic Outdated Duplicate Resolved Low Quality

Fixed & rebased over master, here and in one other place below.

I assume iojs+release is still used?

thefourtheye commented Sep 16, 2015

Copy link
Copy Markdown
Contributor

@rvagg Are all the jobs right? We can fix it now itself

mgol commented Sep 16, 2015

Copy link
Copy Markdown
Contributor Author

The only job in doc/releases.md that have iojs in its name is currently iojs+release; if that was changed to a node-based one, there wouldn't be any more left.

silverwind commented Sep 18, 2015

Copy link
Copy Markdown
Contributor

iojs+release is still the main release job. Can we rename it?

silverwind commented Sep 23, 2015

Copy link
Copy Markdown
Contributor

@mzgol terrible sorry, but it appears I landed one of the changes here in 76c80d4, which breaks this patch. Could you remove the second commit? Also, LGTM.

doc: jenkins-iojs.nodesource.com -> ci.nodejs.org
c7ee4a3 
The Jenkins URL has changed from https://jenkins-iojs.nodesource.com
to https://ci.nodejs.org. The former address now redirects to the
latter. Some jobs has also changed names.

mgol commented Sep 23, 2015 via email

Copy link
Copy Markdown
Contributor Author

silverwind pushed a commit that referenced this pull request Sep 23, 2015
doc: jenkins-iojs.nodesource.com -> ci.nodejs.org
02448c6 
The Jenkins URL has changed from https://jenkins-iojs.nodesource.com
to https://ci.nodejs.org. The former address now redirects to the
latter. Some jobs has also changed names.

PR-URL: #2886
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Roman Reiss <me@silverwind.io>

silverwind commented Sep 23, 2015

Copy link
Copy Markdown
Contributor

Thanks, landed in 02448c6.

silverwind closed this Sep 23, 2015
mgol deleted the io2node branch September 24, 2015 14:49
Fishrock123 pushed a commit that referenced this pull request Sep 25, 2015
doc: jenkins-iojs.nodesource.com -> ci.nodejs.org
75d5dce 
The Jenkins URL has changed from https://jenkins-iojs.nodesource.com
to https://ci.nodejs.org. The former address now redirects to the
latter. Some jobs has also changed names.

PR-URL: #2886
Reviewed-By: Rod Vagg <r@va.gg>
Reviewed-By: Evan Lucas <evanlucas@me.com>
Reviewed-By: Roman Reiss <me@silverwind.io>
This was referenced Sep 30, 2015
Closed
Closed
s4sc mentioned this pull request Nov 30, 2023
Open
AyushG3112 mentioned this pull request Nov 30, 2023
Open
thakursachin467 mentioned this pull request Nov 30, 2023
Open
erdun mentioned this pull request Nov 30, 2023
Open
olegnn mentioned this pull request Nov 30, 2023
Open
RoeeOren mentioned this pull request Nov 30, 2023
Open
GregorFendt mentioned this pull request Nov 30, 2023
Open
timzwit mentioned this pull request Nov 30, 2023
Open
aliceUnhinged613 mentioned this pull request Nov 30, 2023
Open
solebox mentioned this pull request Nov 30, 2023
Open
karencapiiro mentioned this pull request Nov 30, 2023
Open
RickAllMighty8195 mentioned this pull request Feb 11, 2024
Open
Megoy-Tambayan mentioned this pull request Feb 11, 2024
Open
Dixiejane mentioned this pull request Feb 11, 2024
Open
JCsomeShots mentioned this pull request Feb 11, 2024
Open
jj-crypto mentioned this pull request Feb 11, 2024
Open
ZRDK mentioned this pull request Feb 11, 2024
Open
buzzlightyear726 mentioned this pull request Feb 11, 2024
Open
Renukaloch mentioned this pull request Feb 11, 2024
Open
MaxMood96 mentioned this pull request Feb 11, 2024
Open
davidmather mentioned this pull request Feb 11, 2024
Open
mrod0101 mentioned this pull request Feb 11, 2024
Closed
CxsLucyfer mentioned this pull request Feb 11, 2024
Open
dewaynesmith3711 mentioned this pull request Feb 11, 2024
Open
kingjay66 mentioned this pull request Feb 11, 2024
Open
zhangpan0907 mentioned this pull request Feb 11, 2024
Open
Supermajestcd mentioned this pull request Feb 12, 2024
Open
MatheusKael mentioned this pull request Feb 12, 2024
Open
SanchitMahajan236 mentioned this pull request Feb 12, 2024
Open
terrorizer1980 mentioned this pull request Feb 12, 2024
Open
myvali998 mentioned this pull request Feb 12, 2024
Open
ambai mentioned this pull request Feb 12, 2024
Open
dewaynesmith3711 mentioned this pull request Feb 12, 2024
Open
BryanLaura mentioned this pull request Feb 12, 2024
Open
nicholaschoi168 mentioned this pull request Feb 12, 2024
Open
jefeaguila mentioned this pull request Feb 12, 2024
Open
Michaela813 mentioned this pull request Feb 12, 2024
Open
chrismilleruk mentioned this pull request Feb 12, 2024
Open
Michaela813 mentioned this pull request Feb 12, 2024
Open
JoanKalanzi1 mentioned this pull request Feb 12, 2024
Open
BryanLaura mentioned this pull request Feb 12, 2024
Open
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters. Learn more about bidirectional Unicode characters
Show hidden characters
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

doc Issues and PRs related to the documentations.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

7 participants

Back | FazBrowse Home | New Git URL