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 link of gyp as Google code is shutting down #1061

Closed
wants to merge 2 commits into from
Closed

Update link of gyp as Google code is shutting down #1061

wants to merge 2 commits into from

Conversation

PeterDaveHello
Copy link
Member

No description provided.

@gibfahn
Copy link
Member

gibfahn commented Nov 29, 2016

Would https://chromium.googlesource.com/external/gyp or https://gyp.gsrc.io/ be a better starting point for someone who doesn't know what gyp is?

@PeterDaveHello
Copy link
Member Author

Oh cool, would you like me to update it?

@gibfahn
Copy link
Member

gibfahn commented Dec 13, 2016

@PeterDaveHello Yes, sounds good to me. I guess you should use https://gyp.gsrc.io as that seems to be the homepage.

@PeterDaveHello
Copy link
Member Author

@gibfahn thanks, if you don't mind, I just directly updated and committed on GitHub, I think you can squash them directly on GitHub, too, if you don't like it, I can clone and squash them locally or just send a new one, thanks.

@gibfahn
Copy link
Member

gibfahn commented Dec 13, 2016

@PeterDaveHello looks fine to me. This needs a node-gyp contributor to review though.

bnoordhuis pushed a commit that referenced this pull request Jan 17, 2017
PR-URL: #1061
Reviewed-By: Ben Noordhuis <info@bnoordhuis.nl>
Reviewed-By: Gibson Fahnestock <gibfahn@gmail.com>
@bnoordhuis
Copy link
Member

Thanks Peter, landed in ff9a6fa.

@bnoordhuis bnoordhuis closed this Jan 17, 2017
@PeterDaveHello PeterDaveHello deleted the patch-1 branch January 17, 2017 11:06
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants