Skip to content

Add ability to set endpoint_url from AWS_ENDPOINT_URL#195

Merged
pjbull merged 1 commit intomasterfrom
aws-endpoint-url
Jan 26, 2022
Merged

Add ability to set endpoint_url from AWS_ENDPOINT_URL#195
pjbull merged 1 commit intomasterfrom
aws-endpoint-url

Conversation

@pjbull
Copy link
Copy Markdown
Member

@pjbull pjbull commented Jan 26, 2022

Merge changes from #193 into master.

(Using local branch so live endpoint tests also execute.)

)

* Add ability to set endpoint_url from AWS_ENDPOINT_URL env variable. Very useful for localstack while waiting for upstream PR from boto3: boto/boto3#2746

* add test

* rm print

* lint
@github-actions
Copy link
Copy Markdown
Contributor

@codecov
Copy link
Copy Markdown

codecov bot commented Jan 26, 2022

Codecov Report

Merging #195 (2a4e221) into master (3827cd8) will increase coverage by 0.0%.
The diff coverage is 100.0%.

@@          Coverage Diff           @@
##           master    #195   +/-   ##
======================================
  Coverage    94.3%   94.3%           
======================================
  Files          21      21           
  Lines        1197    1198    +1     
======================================
+ Hits         1129    1130    +1     
  Misses         68      68           
Impacted Files Coverage Δ
cloudpathlib/s3/s3client.py 95.1% <100.0%> (+<0.1%) ⬆️

@pjbull pjbull merged commit 749e7d6 into master Jan 26, 2022
@pjbull pjbull deleted the aws-endpoint-url branch January 26, 2022 05:22
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.

2 participants