-
Notifications
You must be signed in to change notification settings - Fork 6
deps: Drop support for Python 3.7 and 3.8 #337
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
Merged
+97
−260
Merged
Changes from all commits
Commits
Show all changes
48 commits
Select commit
Hold shift + click to select a range
9b32164
feat: Drop support for Python 3.7 and 3.8
google-labs-jules[bot] dccef48
Updates python version in lint.yml
chalmerlowe f81c738
Updates owlbot, removing reference to 3.8
chalmerlowe 5cb30ca
Updates CONTRIBUTING.rst
chalmerlowe 7f44424
updates pytest warnings
chalmerlowe 3e4e6f6
Removes test_samples-impl ref to older virtualenv package
chalmerlowe 7beea0d
Removes references to pandas older than 1.5.0
chalmerlowe 32d7b8f
Removes pandas older than 1.5 and misc changes
chalmerlowe e871d73
updates pandas in setup.py
chalmerlowe 2bdf3ef
more updates related to pandas
chalmerlowe d902e31
still broken
chalmerlowe a0543b9
Updates FutureWarning tests to account for unittest coverage
chalmerlowe 6a5d943
Updates json array type tests to account for unittest coverage
chalmerlowe 1bbd5f7
updates python version checks to ensure coverage
chalmerlowe 96a5e97
update json test for unittest coverage
chalmerlowe 8af9bee
Update pandas_backports unittests to ensure coverage
chalmerlowe 78574c0
Updates per review comments
chalmerlowe 9ca34b8
Merge branch 'main' into remove-python-37-38
chalmerlowe de202f4
moves class from version specific compliance file to generic file
chalmerlowe fd9dd27
Removes weird cut and paste error
chalmerlowe bfe9115
fix linting errors
chalmerlowe 91e437e
updates import statement to ensure import of JSONArrowType
chalmerlowe eca1ab7
Revise required github status checks
chalmerlowe 06b4e88
update linting
chalmerlowe d481be4
temporarily marking a class as no cover
chalmerlowe f4a9fc6
more updates
chalmerlowe 7585a86
marked several snippets as pragma no cover
chalmerlowe c777141
updates linting
chalmerlowe 7d745b0
Updates constraints and setup.py
chalmerlowe 9511e7a
migrates class from one time compliance file to another
chalmerlowe 9638e43
updating pyarrow version
chalmerlowe d433fae
Updates linting
chalmerlowe 4c522b3
removes determine all and module reload tests
chalmerlowe cbfcd1a
updates re: ndarrybackedextensionarray
chalmerlowe 2f9fc09
testing blacken as part of owlbot processing using 3.8
chalmerlowe 7b967e7
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] 18098f0
updates blacken to 3.10
chalmerlowe 20e6565
update python version in lint.yml
chalmerlowe f39b97f
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] c23d435
updates owlbot.py
chalmerlowe a52c46d
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] 4ebe910
updates owlbot.py
chalmerlowe cd4942e
testing lint.yml
chalmerlowe 32245de
testing linting issue
chalmerlowe 695c33d
testing linting issue
chalmerlowe 9ea13d3
🦉 Updates from OwlBot post-processor
gcf-owl-bot[bot] 2ee0533
updates lint_setup session
chalmerlowe f59ee72
Update noxfile.py
chalmerlowe File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
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
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
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
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
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
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
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
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
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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,2 +1 @@ | ||
pytest===7.4.4; python_version == '3.7' # prevents dependabot from upgrading it | ||
pytest==8.3.3; python_version > '3.7' | ||
pytest==8.3.5 |
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
Oops, something went wrong.
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.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
From the comments, it seems removing Python 3.7 could ensure pandas >= 1.5.0, and thus ensure JSONArray and JSONDtype to be available. Do we still need
_determine_all()
?There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Removed.