-
Notifications
You must be signed in to change notification settings - Fork 225
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
Automated documentation previews in pull requests #342
Comments
@weiji14 I received your request to install a github app to the pygmt repository, but I can't find any description about the app. Is it related to this issue? Could you post the homepage of this app? |
Here it is https://zeit.co/. |
I'll try to get it to work on my own fork first. Need to find a way to make it do |
I’ve approved and installed the app. |
Cool, this might take a bit of time (and experimentation) to set up, but hope it'll be worth it! |
Hmm, I might have messed up the github app integration. Sorry @seisman, could you uninstall it first, and I'll try to request it to be reinstalled again? |
@weiji14 I've uninstalled it. |
@weiji14 I've approved the app and send you an email with a Now link, which may be necessary for installation. |
Thanks, Wei! I experimented with this in the past but it was hard to do with the CIs because forks don’t have access to encrypted variables needed for deploying. Does this app offer a way around that? |
The Zeit Now for Github App integration does a deploy every time a I did experiment with making deployments via Github Actions CI (which I've attempted in my personal fork at weiji14#45) but that required setting a 'secrets' token variable. We can use a token as well to deploy to Zeit Now, but it's not strictly required with the app. |
Description of the desired feature
Make it easier to preview changes to the documentation online in every pull request, rather than having to clone the branch and do a local build. I.e. Continuous Documentation.
Will be especially helpful for new contributors and even reviewers wanting a quick look at what's going up on the website. This was hinted at during #314.
I'm gonna try to set up an automated Continuous Deployment site, just a heads up!
Are you willing to help implement and maintain this feature? Yes
The text was updated successfully, but these errors were encountered: