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

Use new sybil 3 name for codeblock parser if available #167

Merged
merged 4 commits into from
Feb 25, 2022

Conversation

bnavigator
Copy link
Contributor

@bnavigator bnavigator commented Dec 21, 2021

Hi, we are trying to enable packages for Python 3.10 in openSUSE Tumleweed. We need sybil 3 for that, but not being able to update testfixtures blocks it.

AFAICT the rename of the CodeBlockParser was the only relevant change. Edit: Did not check the doctests. My bad.

New version of #166: Also allow older version for Python 2 support.

@cjw296
Copy link
Member

cjw296 commented Dec 22, 2021

Hi Ben! Not sure what your timescales are, but there's a big new release of testfixtures brewing here.

That already has Sybil 3 in it, along with dropping Python 2 support, a documentation refresh and, hopefully, type annotations throughout.

I don't have fixed timescales on that, but probably early in the new year. Would that work for you?

@bnavigator
Copy link
Contributor Author

Hi,

I was not aware that you prepare a new release in a separate fork. Whatever your release schedule is, the openSUSE package will follow. Until then, in order to keep the openSUSE package ecosystem in a sane state, we will use this patch to have both sybil 3 and testfixtures 6.18.3 coexisting.

I am closing this PR then, as cjw296@81981ef will be the more advanced change for the future release.

@cjw296
Copy link
Member

cjw296 commented Feb 25, 2022

testfixtures v7 is taking longer than I'd like, so let's see if we can land this.

@cjw296 cjw296 reopened this Feb 25, 2022
@cjw296 cjw296 merged commit daa5c39 into simplistix:master Feb 25, 2022
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.

2 participants