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 py2 and remove testtools #279

Merged
merged 2 commits into from Apr 30, 2020
Merged

Drop support for py2 and remove testtools #279

merged 2 commits into from Apr 30, 2020

Conversation

cdent
Copy link
Owner

@cdent cdent commented Apr 30, 2020

The initial motivation for this change was removing the testtools
library. It has a cyclical dependency with the fixutures module
which makes installing gabbi in some environments (notably bazel)
difficult. Please to fix this problem in the testtools and fixtures
communities have gone unaddressed for years so an effort was
undertaken to remove them.

That work exposed that much of what testtools was doing was masking
over differences in python3 and python3 unittests. Since it is 2020,
choosing to drop support for python2 is a legitimate choice. People
who still need support for python2 can use an earlier version of
gabbi.

To signal that change, the version number has been raised to 2.0.0.

tox, travis config and setup.cfg have been updated accordingly.

The initial motivation for this change was removing the testtools
library. It has a cyclical dependency with the fixutures module
which makes installing gabbi in some environments (notably bazel)
difficult. Please to fix this problem in the testtools and fixtures
communities have gone unaddressed for _years_ so an effort was
undertaken to remove them.

That work exposed that much of what testtools was doing was masking
over differences in python3 and python3 unittests. Since it is 2020,
choosing to drop support for python2 is a legitimate choice. People
who still need support for python2 can use an earlier version of
gabbi.

To signal that change, the version number has been raised to 2.0.0.

tox, travis config and setup.cfg have been updated accordingly.
@cdent cdent requested review from FND and jasonamyers April 30, 2020 15:51
pypy3 is considered "supported" by travis now, so we don't need
a specific dist, and the location of placement master has moved.
Fix to the new location and add -L to the curl to save us if
there are future changes.
@cdent cdent merged commit c7dd82a into master Apr 30, 2020
Copy link
Collaborator

@FND FND left a comment

Choose a reason for hiding this comment

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

This looks pretty good AFAICT - good job!

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

2 participants