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

Draft PR for github action #54

Open
wants to merge 38 commits into
base: master
Choose a base branch
from
Open

Conversation

ThomVett
Copy link

No description provided.

@GeoffNN
Copy link
Contributor

GeoffNN commented Mar 24, 2021

This is nice, thanks a lot @ThomVett!

Do you think you can check out #55 for more github action fun? It seems like public repos have unlimited gitub action time, so it would be great if we could build the docs there directly.

@fabianp
Copy link
Member

fabianp commented Mar 24, 2021

that would indeed be awesome!

@GeoffNN
Copy link
Contributor

GeoffNN commented Mar 25, 2021

Right now, the docs are built with sphinx by running make html from within the doc folder. Then running make upload uploads it to our server and refreshes the website.
Hopefully it should be easy to integrate in the Action?

@GeoffNN
Copy link
Contributor

GeoffNN commented Mar 26, 2021

@fabianp I don't have the permissions to access the secret keys to set up the github action for uploading to our server. Could you please add the two secrets required in https://github.com/google-github-actions/setup-gcloud ?
I think we need a new service account; the process is described here

EDIT: Added keys

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