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

use readthedocs for docs builds #21

Open
tonyfast opened this issue Dec 12, 2023 · 1 comment
Open

use readthedocs for docs builds #21

tonyfast opened this issue Dec 12, 2023 · 1 comment

Comments

@tonyfast
Copy link
Member

@bollwyvl mentioned a 😞 experience for docs build with the system in place now. #17 (comment) the pain can be seen in https://github.com/deathbeds/nbconvert-a11y/actions/runs/7160638366/job/19495254744

we should just lean on read the docs.

@bollwyvl
Copy link

GitHub Pages builds are fine, and strictly better for heavy-duty builds which can leverage cache (looking at you, playwright) as long as they have rules in front of them (e.g. only on push/merge to main) so they don't show up in drive-by PRs, I fixed this typo, why is it covered in ✖️ and sending me email?

on: {push: {branches: [main]}}

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

No branches or pull requests

2 participants