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

Deploy docs via GH actions #3046

Closed
lilyminium opened this issue Dec 1, 2020 · 0 comments · Fixed by #3053
Closed

Deploy docs via GH actions #3046

lilyminium opened this issue Dec 1, 2020 · 0 comments · Fixed by #3053

Comments

@lilyminium
Copy link
Member

Is your feature request related to a problem?

Travis is dead to us (#3036), but it's currently how we deploy docs.

Describe the solution you'd like

Deploy docs via GH actions.

Describe alternatives you've considered

I have already tried and failed to use ReadTheDocs due errors that crop up from the structure of the MDA repo (where the normal top-level repo is in "package/"); I will re-investigate but at the time, the RTD build structure wasn't flexible enough to account for this quirk. It would be nice to get an RTD pipeline going even if we don't use it for actual docs, as it can add a check to GH PRs.

Additional context

Follow on from #3040

It would be neat if we could inspect for doc diffs and only deploy when that happens. Not certain if that is possible.

@lilyminium lilyminium self-assigned this Dec 1, 2020
lilyminium added a commit that referenced this issue Dec 6, 2020
Fixes #3046. Deploys docs in gh-ci.yaml script and removes Travis
lilyminium added a commit to lilyminium/mdanalysis that referenced this issue Dec 7, 2020
Fixes MDAnalysis#3046. Deploys docs in gh-ci.yaml script and removes Travis
lilyminium added a commit to lilyminium/mdanalysis that referenced this issue Dec 9, 2020
Fixes MDAnalysis#3046. Deploys docs in gh-ci.yaml script and removes Travis
lilyminium added a commit to lilyminium/mdanalysis that referenced this issue Dec 14, 2020
Fixes MDAnalysis#3046. Deploys docs in gh-ci.yaml script and removes Travis
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants