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

Update all 'iojs' GitHub links to 'nodejs' organization #1715

Closed
wants to merge 3 commits into from
Closed

Update all 'iojs' GitHub links to 'nodejs' organization #1715

wants to merge 3 commits into from

Conversation

fhemberger
Copy link
Contributor

I hope it's ok I went through all the files updating the links, we need at least GOVERNANCE.md and WORKING_GROUPS.md for the io.js website.

@mscdex mscdex added the meta Issues and PRs related to the general management of the project. label May 16, 2015
@silverwind
Copy link
Contributor

Looking good on first glance. If you wouldn't mind, could you also update the gitter links so we have these changes contained in one commit?

@Fishrock123 Fishrock123 added the doc Issues and PRs related to the documentations. label May 16, 2015
@silverwind
Copy link
Contributor

There are a few trailing whitespace issues:

Applying Pull Request from: https://github.com/iojs/io.js/pull/1715.patch
Applying: Update all 'iojs' GitHub links to 'nodejs' organization
/Users/silverwind/git/iojs-merge/.git/rebase-apply/patch:4314: trailing whitespace.
* @chrisdickinson nomination to TC https://github.com/nodejs/io.js/issues/51
/Users/silverwind/git/iojs-merge/.git/rebase-apply/patch:4315: trailing whitespace.
* Move readable-stream to iojs org and make authoritative source for io.js to pull _from_ https://github.com/nodejs/io.js/issues/55
/Users/silverwind/git/iojs-merge/.git/rebase-apply/patch:4316: trailing whitespace.
* Deprecate domains https://github.com/nodejs/io.js/issues/66
/Users/silverwind/git/iojs-merge/.git/rebase-apply/patch:4317: trailing whitespace.
* Decide how to go forward with caine https://github.com/nodejs/io.js/issues/117
/Users/silverwind/git/iojs-merge/.git/rebase-apply/patch:4325: trailing whitespace.
* Statement / stance from TC on exposing a Promises API https://github.com/nodejs/io.js/issues/11
warning: squelched 9 whitespace errors
warning: 14 lines add whitespace errors.

@fhemberger
Copy link
Contributor Author

Sure, will update gitter and check the whitespace issues later.

@fhemberger
Copy link
Contributor Author

Whitespace in 2014-12-10.md should be fine now (only the issue links should have trailing whitespaces), Gitter links are updated as well.

@silverwind
Copy link
Contributor

Right I totally forgot about the whitespace significance. LGTM

silverwind pushed a commit that referenced this pull request May 18, 2015
Replaced all links in doc and comments to point to the new org.

PR-URL: #1715
Reviewed-By: Roman Reiss <me@silverwind.io>
@silverwind
Copy link
Contributor

Landed with a conforming commit message at 0662747

@silverwind silverwind closed this May 18, 2015
@rvagg rvagg mentioned this pull request May 23, 2015
andrewdeandrade pushed a commit to andrewdeandrade/node that referenced this pull request Jun 3, 2015
Replaced all links in doc and comments to point to the new org.

PR-URL: nodejs/node#1715
Reviewed-By: Roman Reiss <me@silverwind.io>
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. meta Issues and PRs related to the general management of the project.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants