Skip to content

ci: fixing baseurl #395

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 5, 2025
Merged

ci: fixing baseurl #395

merged 4 commits into from
Feb 5, 2025

Conversation

henryiii
Copy link
Member

@henryiii henryiii commented Feb 5, 2025

Incorrect baseurl calculation.

@eduardo-rodrigues
Copy link
Member

Thanks for trying this out, Henry! Do we need to wait for this PR to get merged to then get a preview in a new PR? Guessing so :).

@henryiii
Copy link
Member Author

henryiii commented Feb 5, 2025

No, you can click on the readthedocs "check" below and see the preview. But the URLs are messed up so things like the css doesn't load, so working that out.

@henryiii henryiii force-pushed the henryiii/ci/rtd-fix branch 3 times, most recently from 79d5b33 to 1fb9282 Compare February 5, 2025 17:04
Signed-off-by: Henry Schreiner <[email protected]>
@henryiii henryiii force-pushed the henryiii/ci/rtd-fix branch 2 times, most recently from 6d63e12 to 59fd488 Compare February 5, 2025 17:18
@eduardo-rodrigues
Copy link
Member

The homepage is now previewed nicely - cool! - but all left-menu links (at least, did not check everything) give a 404. There's still some issue with the pages, I'm afraid.

@henryiii henryiii force-pushed the henryiii/ci/rtd-fix branch from aba3755 to 0a81ca7 Compare February 5, 2025 20:41
@henryiii henryiii force-pushed the henryiii/ci/rtd-fix branch from 5eb8f5c to 936b541 Compare February 5, 2025 20:48
@henryiii
Copy link
Member Author

henryiii commented Feb 5, 2025

I think it's fixed now.

@henryiii
Copy link
Member Author

henryiii commented Feb 5, 2025

I'll go ahead and merge so I can check (and fix if needed) the main site. Local builds and PR previews seem fine.

@henryiii henryiii merged commit d38795b into main Feb 5, 2025
2 checks passed
@henryiii henryiii deleted the henryiii/ci/rtd-fix branch February 5, 2025 20:55
@eduardo-rodrigues
Copy link
Member

Hey. Many thanks!

All looks good except the rendering of https://scikit-hepgithubio--395.org.readthedocs.build/en/395/packages/qr_codes/, which shows links but not the figures rendered. This being said, all seems fine at https://scikit-hep.org/packages/qr_codes/, actually.

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