search

bavovanachte / sphinx-wavedrom

A sphinx extension that allows including wavedrom diagrams by using its text-based representation
MIT License
32 stars 18 forks source link

Fix removed ENOENT of sphinx.util.osutil in sphinx 4 #32

Closed krmarien closed 3 years ago

krmarien commented 3 years ago

This package is not compatible with the latest version of sphinx.

ENOENT was removed in sphinx 4.

bavovanachte commented 3 years ago

Thanks for the PR!

Do you need this urgently? Your PR brings to my attention that I need to test for different sphinx versions in CI, and I'd like to put that in place first before merging. However, if it's urgent I can do some manual testing with older sphinx versions

krmarien commented 3 years ago

I do not need it urgently, so take your time to update the CI flow

bavovanachte commented 3 years ago

I've added sphinx version testing in #33 . I've tried adding 4.* to your fork, but I'm confused on how the github actions work with forks. I'll just merge and let the CI run on master. If there are issues I can fix them or revert the PR

Thanks again for the contribution!

bavovanachte commented 3 years ago

Released in https://github.com/bavovanachte/sphinx-wavedrom/releases/tag/3.0.1