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

Fix docs gha #177

Merged
merged 13 commits into from
May 8, 2022
Merged

Fix docs gha #177

merged 13 commits into from
May 8, 2022

Conversation

ocefpaf
Copy link
Collaborator

@ocefpaf ocefpaf commented May 6, 2022

Also, testing with nbsphix, not ready for review yet.

@ocefpaf
Copy link
Collaborator Author

ocefpaf commented May 6, 2022

The docs are building and the push to gh-pages, from a PR, is failing as expected.

@ocefpaf
Copy link
Collaborator Author

ocefpaf commented May 6, 2022

Docs are build and the failed linkcheck is expected b/c the file will be on that URL after the merge of this PR. See https://github.com/python-windrose/windrose/runs/6324143101?check_suite_focus=true#step:6:91

@ocefpaf ocefpaf marked this pull request as ready for review May 6, 2022 14:15
@@ -2,8 +2,7 @@ name: Build and Deploy docs

on:
push:
branches:
- [master, main]
branches: [master, main]
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@scls19fr this should fix the error that is happening on master.

@@ -14,7 +14,7 @@ Welcome to windrose's documentation!
:caption: Contents:

Install <install.rst>
Usage <usage.rst>
Usage <usage-output.ipynb>
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

This PR changes how we this part of the documentation. Instead of a rst file we have a notebook that is run at build time, to ensure it is working. That reduces the number of files we need to sync (screenshots, instructions in multiple places, etc) and make it easier for users b/c they can download the notebook and run the docs if they want to follow along.

@ocefpaf ocefpaf mentioned this pull request May 6, 2022
@s-celles s-celles merged commit 698b126 into python-windrose:master May 8, 2022
@ocefpaf ocefpaf deleted the fix_docs_gha branch May 8, 2022 14:16
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.

2 participants