Skip to content

Commit

Permalink
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
c
Browse files Browse the repository at this point in the history
nameexhaustion committed Oct 18, 2024
1 parent eaaec47 commit bb78ff4
Showing 2 changed files with 2 additions and 3 deletions.
2 changes: 1 addition & 1 deletion py-polars/polars/io/cloud/credential_provider.py
Original file line number Diff line number Diff line change
@@ -55,7 +55,7 @@ class CredentialProviderAWS(CredentialProvider):
at any point without it being considered a breaking change.
"""

def __init__(self, *, profile_name: str = "default") -> None:
def __init__(self, *, profile_name: str | None = None) -> None:
"""
Initialize a credential provider for AWS.
3 changes: 1 addition & 2 deletions py-polars/tests/unit/io/cloud/test_cloud.py
Original file line number Diff line number Diff line change
@@ -33,9 +33,8 @@ def raises(*_: None, **__: None) -> None:

monkeypatch.setattr(pl.CredentialProviderAWS, "__init__", raises)

# By default `credential_provider="auto"`
with pytest.raises(AssertionError, match=err_magic):
pl.scan_parquet("s3://bucket/path")
pl.scan_parquet("s3://bucket/path", credential_provider="auto")

# Passing `None` should disable the automatic instantiation of
# `CredentialProviderAWS`

0 comments on commit bb78ff4

Please sign in to comment.