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

Adjust the config loader import to support towncrier >= 22.8.0rc1 #61

Merged
merged 4 commits into from
Aug 31, 2022

Conversation

ericpre
Copy link
Contributor

@ericpre ericpre commented Aug 29, 2022

This patch implements compatibility with the newly released towncrier >= 22.8.0rc1. Since the extension relies on the Towncrier's private API, this sort of thing is bound to happen time to time. It is solved by using a fallback import logic.

This change also adds the said Towncrier version to the CI matrix.

Fixes #60.

@ericpre
Copy link
Contributor Author

ericpre commented Aug 30, 2022

@webknjaz, does this PR look good to you? Thanks! :)

@webknjaz
Copy link
Member

It seems fine but I don't like reviewing from phone because I may miss the bigger picture so I'll probably get to it tomorrow when I have access to my laptop. I won't be cutting a release from phone anyway. Plus, the next release will likely wait for the other two open PRs to be complete first.
One improvement that seems necessary is adding this new release to the GHA matrix so that it's visible they it works as expected.

@ericpre
Copy link
Contributor Author

ericpre commented Aug 30, 2022

Thank you for the quick follow up - this sounds good to me!

@webknjaz webknjaz changed the title Fix import for towncrier >= 22.8.0 Adjust the config loader import to support towncrier >= 22.8.0rc1 Aug 31, 2022
@webknjaz webknjaz merged commit f6e314a into sphinx-contrib:master Aug 31, 2022
@ericpre
Copy link
Contributor Author

ericpre commented Sep 1, 2022

Thank you!

@webknjaz
Copy link
Member

webknjaz commented Sep 4, 2022

This has been released in v0.3.0a0: https://pypi.org/project/sphinxcontrib-towncrier/0.3.0a0/.

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.

towncrier 22.8.0rc1 breaks sphinxcontrib-towncrier
2 participants