Skip to content

CI: Remove docs build from pipelines #32074

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

Merged
merged 4 commits into from
Feb 26, 2020
Merged

CI: Remove docs build from pipelines #32074

merged 4 commits into from
Feb 26, 2020

Conversation

datapythonista
Copy link
Member

@datapythonista datapythonista commented Feb 18, 2020

At the moment we're building the web and the docs from both Actions and Pipelines. While is still under discussion where things should be hosted, it's probably worth to just build on Actions for now, and publish from there to both the OVH cloud, and to GitHub pages.

This needs the GitHub pages ssh key to be added to the secrets of this repo with name "github_pages_ssh_key".* Can someone with permissions please add it? Or if we already got it with a different name, please let me know the name.

@datapythonista datapythonista added Docs CI Continuous Integration labels Feb 18, 2020
@jbrockmendel
Copy link
Member

Can someone with permissions please add it?

cc @TomAugspurger

@TomAugspurger
Copy link
Contributor

I'm not sure what the value of the key is. Perhaps @jorisvandenbossche has it?

@datapythonista
Copy link
Member Author

Sent them via email.

@TomAugspurger
Copy link
Contributor

Thanks @datapythonista. Added to the pandas-dev/pandas github repo's secrets with the name github_pages_ssh_key.

@datapythonista
Copy link
Member Author

Cool, this should be ready then. As a follow up, I'll synchronize the website and the dev docs in the rackspace server in the GitHub Actions build.

@jorisvandenbossche
Copy link
Member

I just noticed https://github.com/ammaraskar/sphinx-action (a github action to build sphinx docs and puts errors inline), might be interesting (but didn't really look at it)

Copy link
Member

@WillAyd WillAyd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm - can you merge master one more time to be sure?

@datapythonista
Copy link
Member Author

All green after rebasing, merging.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI Continuous Integration Docs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

CI: Move documentation build to GitHub actions
5 participants