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

update sphinx comparison #291

Draft
wants to merge 14 commits into
base: develop
Choose a base branch
from
Draft

update sphinx comparison #291

wants to merge 14 commits into from

Conversation

denis-yuen
Copy link
Member

@denis-yuen denis-yuen commented Aug 21, 2024

Description
Update sphinx and transitive dependencies

Issue
See https://ucsc-cgl.atlassian.net/browse/SEAB-6642

Please make sure that you've checked the following before submitting your pull request. Thanks!

@denis-yuen denis-yuen self-assigned this Aug 21, 2024
@denis-yuen
Copy link
Member Author

Ah,

Ignored the following versions that require a different python version: 1.0.0 Requires-Python >=3.10; 8.0.0 Requires-Python >=3.10; 8.0.0rc1 Requires-Python >=3.10; 8.0.1 Requires-Python >=3.10; 8.0.2 Requires-Python >=3.10

@denis-yuen denis-yuen changed the title Feature/experiment update sphinx comparison Aug 21, 2024
@denis-yuen
Copy link
Member Author

@aofarrel
Copy link
Collaborator

aofarrel commented Aug 21, 2024

Updating these dependencies in particular can be tricky as it sometimes silently breaks formatting that can get missed at first glance. Will need to check the following before merging:

  • pages generated from markdown (.md)
  • pages with bullet points <-- this one in particular caused issues previously
  • internal and external links
  • the clickable images on the homepage
  • tables

If you run into any issues with glossarpy, ping me so I can make sure it remains up-to-spec!

@denis-yuen
Copy link
Member Author

Thanks for the pointers, will look for those.
Think we'll keep this open till early in 1.17 so we have some extra time to look

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