Uh oh!
There was an error while loading. Please reload this page.
PYTHON-1289: remove unittest2 - #1117
Merged
Merged
Conversation
added 3 commits
November 22, 2021 14:41
absurdfarce
commented
Nov 23, 2021
ContributorAuthor
test_authentication_misconfiguration:MisconfiguredAuthenticationTests passes locally for Python 2.7 so this looks to be another red herring. |
Orenef11
commented
Nov 23, 2021
Contributor
LGTM. |
fruch added a commit
to fruch/python-driver
that referenced
this pull request
Aug 2, 2022
…to sync_with_upstream * 'master' of https://github.com/datastax/python-driver: Merge pull request apache#1126 from eamanu/fix-typos PYTHON-1294: Upgrade importlib-metadata to a much newer version Add tests for recent addition of execution profile support to cassandra.concurrent Merge pull request apache#1122 from andy-slac/concurrent-execution-profiles Merge pull request apache#1119 from datastax/python-1290 Merge pull request apache#1117 from datastax/remove_unittest2 Removing file unexpectedly included in previous PR Merge pull request apache#1114 from haaawk/stream_ids_fix Merge pull request apache#1116 from Orenef11/fix_default_argument_value Comment update following off of apache#1110 Merge pull request apache#1103 from numberly/fix_empty_paging Merge pull request apache#1103 from psarna/fix_deprecation_in_tracing Fixes to the Travis build. (apache#1111)
fruch added a commit
to fruch/python-driver
that referenced
this pull request
Aug 2, 2022
…to sync_with_upstream_2 * 'master' of https://github.com/datastax/python-driver: Merge pull request apache#1126 from eamanu/fix-typos PYTHON-1294: Upgrade importlib-metadata to a much newer version Add tests for recent addition of execution profile support to cassandra.concurrent Merge pull request apache#1122 from andy-slac/concurrent-execution-profiles Merge pull request apache#1119 from datastax/python-1290 Merge pull request apache#1117 from datastax/remove_unittest2 Removing file unexpectedly included in previous PR Merge pull request apache#1114 from haaawk/stream_ids_fix Merge pull request apache#1116 from Orenef11/fix_default_argument_value Comment update following off of apache#1110 Merge pull request apache#1103 from numberly/fix_empty_paging Merge pull request apache#1103 from psarna/fix_deprecation_in_tracing Fixes to the Travis build. (apache#1111)
datasttax referenced
this pull request
in datasttax/python-driver
Jul 28, 2023
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for freeto join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Motivated by recent PR from @Orenef11 (#1115). After digging into it a bit it became clearer that unittest2 likely isn't adding any value at this point. It was created as a backport for new functionality in the unittest package of 2.7 to earlier versions in the 2.x line. None of those earlier versions are supported by the current driver, however, and all 3.x Python versions supported by unittest2 are also now EOL. Seems ideal to remove the dependency all together as long as we can get to a clean test run without it.