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

Fix compatibility with Sphinx 4 and 5.0b1. #1

Merged
merged 2 commits into from
May 18, 2022
Merged

Fix compatibility with Sphinx 4 and 5.0b1. #1

merged 2 commits into from
May 18, 2022

Conversation

icemac
Copy link

@icemac icemac commented May 17, 2022

Hi @stevepiercy, this PR adds up on the changes you made to repoze.sphinx.autointerface. Locally the tests run with Sphinx 4, 4.5 and 5.0b1.

Spoiler: I am not sure if it is right what I did, I just made the things necessary to get the tests running again.

Could you please consider this PR to make some progress for this package.

@@ -88,7 +88,7 @@ def keyfunc(entry):
self.add_line(u'', '<autointerface>')
self.indent += self.content_indent
sourcename = u'docstring of %s.%s' % (self.fullname, name)
docstrings = [prepare_docstring(doc, None)]
Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is needed for version 5.0b1

@stevepiercy
Copy link
Owner

@icemac thank you! I also have no idea if it is the right thing, but as long as tests pass, I'm happy.

@stevepiercy
Copy link
Owner

Test environments are out of date and cause failures. We should drop those that are now EOL, including 3.6.

@icemac
Copy link
Author

icemac commented May 18, 2022

Hi @stevepiercy I updated the GHA config, so it should be able to run through.

@stevepiercy stevepiercy merged commit ae7ce32 into stevepiercy:sphinx-40-compat May 18, 2022
@stevepiercy
Copy link
Owner

All green! Thank you!

@icemac icemac deleted the sphinx-40-compat branch May 19, 2022 05:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
2 participants