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 readthedocs formatting issues #1095

Merged
merged 3 commits into from
Sep 9, 2024
Merged

Conversation

aphearin
Copy link
Contributor

@aphearin aphearin commented Sep 9, 2024

As noticed by @pllim a recent PR (probably #1093) broke the readthedocs formatting. This PR attempts to fix.

@pllim
Copy link
Member

pllim commented Sep 9, 2024

Nope... See rendered doc at https://halotools--1095.org.readthedocs.build/en/1095/

@pllim
Copy link
Member

pllim commented Sep 9, 2024

Maybe you are not using the theme that you think you are using. Raw log might give you more info. Example: https://readthedocs.org/api/v2/build/25571238.txt

@pllim
Copy link
Member

pllim commented Sep 9, 2024

BTW, if you want to save CPU cycles for CI, you can put [ci skip] on your commit message. Actions respects that and won't run. RTD does not, so it would run anyway.

@aphearin aphearin merged commit f1a6401 into astropy:master Sep 9, 2024
1 check passed
@aphearin aphearin deleted the docs_bugs branch September 9, 2024 20:32
@aphearin aphearin restored the docs_bugs branch September 9, 2024 20:43
@aphearin aphearin deleted the docs_bugs branch September 9, 2024 20:43
@pllim
Copy link
Member

pllim commented Sep 9, 2024

Looks good now. Glad you figured it out! 🎉

https://halotools.readthedocs.io/en/latest/

@aphearin
Copy link
Contributor Author

aphearin commented Sep 9, 2024

Yes! Mysteriously, it was just this one-liner: e94ce2a. Thanks again for all the help @pllim!

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