admin icon indicating copy to clipboard operation
admin copied to clipboard

Set up Netlify for Pull Request previews for nodejs.org repo

Open XhmikosR opened this issue 6 years ago • 19 comments
trafficstars

Or something else that has the same effect.

Currently one has to pull down the changes locally to see them in action which is far from optimal.

XhmikosR avatar Aug 25 '19 07:08 XhmikosR

@Trott can we do this please? It will really help with PRs. I can help setting it up after you add it to the repo.

XhmikosR avatar Oct 04 '19 12:10 XhmikosR

@Trott can we do this please? It will really help with PRs. I can help setting it up after you add it to the repo.

You can either open a separate issue in the nodejs/admin repo, or I can transfer this issue there.

Trott avatar Oct 04 '19 16:10 Trott

@Trott yeah, please move it where appropriate. I thought it belonged to this repo.

To clarify, I don't mind whatever solution we go with, we just need this to make our lives easier :)

XhmikosR avatar Oct 06 '19 14:10 XhmikosR

@nodejs/tsc @nodejs/community-committee @nodejs/website

Trott avatar Oct 08 '19 02:10 Trott

Couldn't agree more, would love to have previews rather than being forced to run locally and/or ask for screenshots.

SGTM

phillipj avatar Oct 08 '19 06:10 phillipj

Something new since I made this issue. Netlify has changed their pricing. Not sure if it will affect us, just throwing it out there.

In theory, we should be OK, at least for starters.

XhmikosR avatar Oct 08 '19 06:10 XhmikosR

What are the permissions asked by Netlify? Is it an OAuth or a GitHub app ?

targos avatar Oct 08 '19 06:10 targos

https://www.netlify.com/docs/github-permissions/

XhmikosR avatar Oct 08 '19 06:10 XhmikosR

LGTM

targos avatar Oct 08 '19 06:10 targos

I guess we need to setup a Netlify account for the org. Should it belong to the Build WG?

targos avatar Oct 08 '19 06:10 targos

https://github.com/nodejs/nodejs.dev has some sort of Google Cloud Build staging thing going on. Could that potentially be used? I think @MylesBorins set it up.

richardlau avatar Oct 08 '19 16:10 richardlau

@richardlau TBH the setup that we have on nodejs.dev is not neccessarily ideal for everyone and is a little bit more hands on then simply installing netlify.

MylesBorins avatar Oct 08 '19 16:10 MylesBorins

What about https://github.com/nodejs/admin/issues/422#issuecomment-539369086 ?

I guess we need to setup a Netlify account for the org. Should it belong to the Build WG?

targos avatar Oct 23 '19 14:10 targos

I guess we need to setup a Netlify account for the org. Should it belong to the Build WG?

I think it should be more general. since all our doc service may need it.

gengjiawen avatar Apr 01 '20 03:04 gengjiawen

The Build WG currently manages all Node secrets, so yes, I would say it's the build wg.

mcollina avatar Apr 01 '20 07:04 mcollina

@nodejs/build

Trott avatar Apr 03 '20 18:04 Trott

I think sending a PR to https://github.com/nodejs-private/secrets to store the credentials would be enough, people who place their public keys in the corresponding folder there would just get the access (but then yes currently only build and a few others have read access to that repo in the first place)

joyeecheung avatar Apr 04 '20 08:04 joyeecheung

@joyeecheung I would be happy to send that PR if needed, pending getting the details from the Website WG.

bnb avatar Apr 28 '20 17:04 bnb

I just got previews setup on nodejs.dev using GitHub actions, the nodejs-infrastructure gcp account, and a free tier cloudflare cdn.

I could likely spin this up to do staging previews for .org rather painlessly if folks are interested. With that said it isn't a huge deal to use netlify

MylesBorins avatar Apr 28 '20 17:04 MylesBorins

I think this can be closed as we adopted Vercel for previews...

ovflowd avatar Jun 01 '23 10:06 ovflowd