Skip to content
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

Azure blob storage #2284

Merged
merged 4 commits into from
Sep 2, 2021
Merged

Azure blob storage #2284

merged 4 commits into from
Sep 2, 2021

Conversation

romasku
Copy link
Contributor

@romasku romasku commented Sep 1, 2021

No description provided.

@lgtm-com
Copy link

lgtm-com bot commented Sep 1, 2021

This pull request introduces 1 alert when merging afc3076 into e2185f0 - view on LGTM.com

new alerts:

  • 1 for Non-callable called

@codecov
Copy link

codecov bot commented Sep 1, 2021

Codecov Report

Merging #2284 (96ce630) into master (e2185f0) will increase coverage by 11.40%.
The diff coverage is 82.52%.

Impacted file tree graph

@@             Coverage Diff             @@
##           master    #2284       +/-   ##
===========================================
+ Coverage   72.51%   83.92%   +11.40%     
===========================================
  Files          69       69               
  Lines       11282    11375       +93     
  Branches     1739     1748        +9     
===========================================
+ Hits         8181     9546     +1365     
+ Misses       2830     1475     -1355     
- Partials      271      354       +83     
Flag Coverage Δ
cli 63.64% <ø> (ø)
e2e 64.36% <82.52%> (?)
sdk 83.49% <25.24%> (-1.17%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
neuro-sdk/setup.py 0.00% <ø> (ø)
neuro-sdk/src/neuro_sdk/buckets.py 78.14% <82.52%> (+24.04%) ⬆️
neuro-sdk/src/neuro_sdk/admin.py 97.38% <0.00%> (+0.65%) ⬆️
neuro-sdk/src/neuro_sdk/parsing_utils.py 96.85% <0.00%> (+1.04%) ⬆️
neuro-cli/src/neuro_cli/parse_utils.py 91.09% <0.00%> (+1.04%) ⬆️
neuro-sdk/src/neuro_sdk/config_factory.py 94.23% <0.00%> (+1.28%) ⬆️
neuro-sdk/src/neuro_sdk/storage.py 93.72% <0.00%> (+1.39%) ⬆️
neuro-cli/src/neuro_cli/formatters/disks.py 98.57% <0.00%> (+1.42%) ⬆️
neuro-sdk/src/neuro_sdk/url_utils.py 90.90% <0.00%> (+1.81%) ⬆️
neuro-cli/src/neuro_cli/stats.py 83.69% <0.00%> (+2.17%) ⬆️
... and 32 more

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e2185f0...96ce630. Read the comment docs.

@lgtm-com
Copy link

lgtm-com bot commented Sep 2, 2021

This pull request introduces 1 alert when merging 96ce630 into e2185f0 - view on LGTM.com

new alerts:

  • 1 for Non-callable called

@romasku romasku merged commit 702098b into master Sep 2, 2021
@romasku romasku deleted the rs/azure-blob-storage branch September 2, 2021 11:19
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