-
-
Notifications
You must be signed in to change notification settings - Fork 867
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
Adds support for already gzipped content to S3 storage to the boto3 backend #263 #264
Conversation
Hello, the code changes looks okay but the test fails. The way boto3 adds the data is not the same as the way boto does, fix that and then I can merge. |
Codecov Report
@@ Coverage Diff @@
## master #264 +/- ##
==========================================
+ Coverage 72.48% 72.83% +0.34%
==========================================
Files 10 10
Lines 1381 1384 +3
==========================================
+ Hits 1001 1008 +7
+ Misses 380 376 -4
Continue to review full report at Codecov.
|
Hi, sorry took me so long but I got busy and forgot about the PR. |
Don't worry about that part. Test coverage is something that I need to work on in general. Looks good, thanks! |
…ackend #263