-
Notifications
You must be signed in to change notification settings - Fork 31
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
S3 store and resource additions #635
Conversation
…ialsproject/maggma into enhancement/api_s3_changes
Codecov Report
@@ Coverage Diff @@
## main #635 +/- ##
==========================================
- Coverage 89.46% 88.96% -0.50%
==========================================
Files 40 41 +1
Lines 2828 2891 +63
==========================================
+ Hits 2530 2572 +42
- Misses 298 319 +21
Continue to review full report at Codecov.
|
This pull request introduces 3 alerts when merging 1f88339 into c393da8 - view on LGTM.com new alerts:
|
This PR:
S3URLResource
for serving up pre-signed object URLs in S3