Closed drammock closed 2 months ago
hmm, all the pytest jobs pass now, but the build-site jobs all fail with:
Theme error:
setting ablog.inject_templates_after_theme occurs in none of the searched theme configs
not sure if it's related or coincidence, will look tommorow if possible unless someone else has time to look first.
See https://github.com/sunpy/ablog/issues/277 for the error message.
Right so for now we might want to pin our Sphinx version until there is a fix in Ablog
for >= 7.3
🤔 So the last two changes fix our CI for docs and the Read the docs build but, of course, break when we try and install the "dev" Sphinx version from GitHub https://github.com/pydata/pydata-sphinx-theme/actions/runs/8741568491/job/23987894819?pr=1772#step:5:76
🤔 So the last two changes fix our CI for docs and the Read the docs build but, of course, break when we try and install the "dev" Sphinx version from GitHub https://github.com/pydata/pydata-sphinx-theme/actions/runs/8741568491/job/23987894819?pr=1772#step:5:76
One way to get the CI to pass is to revert the pin in pyproject.toml
, and add an extra step in the doc build workflow to downgrade sphinx after the initial package install is done. But: we do kinda need the pin in pyproject.toml
to avoid breaking user sites that use ablog
so I guess we either:
Sorry about ablog, I am in the process of patching it and doing a release when/if my CI passes.
Edit: I have tagged the release, it should be out on pypi shortly
Ok I can try with the upcoming tag in Ablog.
Actually testing against dev helps identify breaking changes against Sphinx. So it served it's purpose even if it gave us a bit of a headache.
As I am working on the CI improvements I see a path here:
pyproject.toml
- this way we ensure we do not break things for users (might need to do this for other dependencies too to avoid compatibility issues moving forward)WDYT @drammock
Yeah a periodic bump test sounds good. Thanks @trallard
But so we then keep in the failing dev test? Or hope/ wait until it works with new ablog version
I am logging off but I can try with the newest Ablog release tomorrow or try a quick fix to uninstall/install the Dev sphinx version while the bigger CI overhaul gets finished
There is both a new version of sphinx that will temporarily fix the crash (though it might affect ablog's configuration), and a blog that will work if sphinx ever raise an error again.
OK everything is passing except CodeCov upload now! @trallard @Carreau ready for review/merge.
should fix CI failures appearing in e.g. #1770