-
Notifications
You must be signed in to change notification settings - Fork 32
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
Add release notes for 2.0.1 #108
Conversation
Hit submit too soon by accident... @sigmavirus24 I wasn't sure what to put for the date and assumed that you'd set it correctly and change the version in the package meta-data. If you want me do do something else here, please let me know. |
I'm not sure what to do about the docs build failure as I don't know what an "intersphinx mapping" is and clicking through the link in the error log did not clarify things 😅 😬 |
@sigmavirus24 or anyone, any update on this? I am happy to do something about the docs check failure if anyone can give me a pointer on how to debug "intersphinx mapping". |
@handrews that would be helpful. The relevant configuration is Line 170 in bd81543
It seems the format has changed per the message
I suspect the documentation liink explains how to fix it. Namely, I'd guess the first example in the docs:
Is something like what we'd want |
8d33e09
to
7355ef4
Compare
intersphinx-mapping needed adjustment, plus there is some sort of incompatibility between sphinx 7.0.0 and sphinx-rtd-theme, so pin sphinx below that point. Allegedly it is fixed in more recent sphinx-rtd-theme installations, but we do not pin that package and it still breaks, so punting that to be solved some other time.
22377c2
to
a25c060
Compare
OK, this should be good to go. My apologies for not noticing the sphinx conf matching the error - for some reason Sphinx routinely frustrates me so much that I miss the obvious every time something goes wrong with it. |
Addresses #107