Conversation
victorlin
force-pushed
the
victorlin/update-dependencies
branch
from
b0e88ba to
a4fea46
Compare
joverlee521
reviewed
Jan 5, 2026
This was referenced Jan 6, 2026
victorlin
force-pushed
the
victorlin/update-dependencies
branch
from
a4fea46 to
2e3e9df
Compare
victorlin
force-pushed
the
victorlin/update-dependencies
branch
from
2e3e9df to
5ee1952
Compare
victorlin
force-pushed
the
victorlin/update-dependencies
branch
from
5ee1952 to
6fe753c
Compare
fsspec was initially added without the [s3] extra due to its recent addition.¹ It's been around for nearly 5 years now, so it should be safe. In addition, the boto3 dependency can be resolved without issues by newer versions of pip. Our minimum pip version target is whatever is bundled with Python 3.8, checked via CI. ¹ "Declare dependency on s3fs since we use fsspec's S3 support" (d1e54c3)
Version 1 was implicitly resolved with the s3fs[boto3] pin removed in the previous commit. It must be explicitly pinned since version 2 affects usage in two ways: 1. It renames ObjectProxy to BaseObjectProxy. 2. It adds type stubs, but only for Python >=3.10. Since Pyright is configured to target 3.8, it would error with v2 upon finding a stubs file without a type definition for the import. https://wrapt.readthedocs.io/en/latest/changes.html#version-2-0-0
Version 2 renames ObjectProxy to BaseObjectProxy. It also adds a type stubs file, but only defines types for Python >=3.10. Since Pyright is configured to target 3.8, a comment is added to ignore the error caused by lack of type definitions. https://wrapt.readthedocs.io/en/latest/changes.html#version-2-0-0
victorlin
force-pushed
the
victorlin/update-dependencies
branch
from
6fe753c to
87ad32f
Compare
joverlee521
approved these changes
Mar 26, 2026
Contributor
joverlee521
left a comment
joverlee521
left a comment
There was a problem hiding this comment.
Thanks for following up on the various dependency pins!
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 free
to 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.
Description of proposed changes
This PR improves dependency declarations for fsspec, boto3, and wrapt. See commits for details.
Related issue(s)
Closes #496
Checklist