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

Drop support for Python < 3.7 #118

Merged
merged 2 commits into from
Dec 23, 2022

Conversation

cjwatson
Copy link
Contributor

zopefoundation/zope.testing#45 broke zope.schema's tests, so I guess it's time to do the corresponding removal here as well.

@icemac icemac merged commit afa7e25 into zopefoundation:master Dec 23, 2022
@icemac
Copy link
Member

icemac commented Dec 23, 2022

Thank you for (reviewing) this PR. 😃

@cjwatson cjwatson deleted the drop-legacy-python branch December 23, 2022 18:43
@cjwatson
Copy link
Contributor Author

Would anyone be willing to either release this to PyPI, or give me access so I can do it myself?

@mgedmin
Copy link
Member

mgedmin commented Jan 1, 2023

Released!

(I don't mind giving you PyPI access, but please mention your PyPI username in your request, as I'd hate to guess and get it wrong.)

@jugmac00
Copy link
Member

jugmac00 commented Jan 1, 2023

Colin on PyPI
https://pypi.org/user/cjwatson/
(He's my colleague at work).

@cjwatson
Copy link
Contributor Author

cjwatson commented Jan 2, 2023

What Jürgen said. This would probably still be useful for future releases. (Thanks for doing 7.0.0.)

cjwatson added a commit to cjwatson/zope.schema that referenced this pull request Jan 2, 2023
I introduced this regression in
zopefoundation#118 by misreading the
try/except/else logic in `zope.schema._messageid`.  The new tests should
guard against somebody else making a similar mistake in future.
cjwatson added a commit to cjwatson/zope.schema that referenced this pull request Jan 2, 2023
I introduced this regression in
zopefoundation#118 by misreading the
try/except/else logic in `zope.schema._messageid`.  The new tests should
guard against somebody else making a similar mistake in future.
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.

None yet

5 participants