Skip to content

[WB-4519] Add ability to delete files from API#1812

Merged
raubitsj merged 3 commits intomasterfrom
api/delete-file
Feb 8, 2021
Merged

[WB-4519] Add ability to delete files from API#1812
raubitsj merged 3 commits intomasterfrom
api/delete-file

Conversation

@annirudh
Copy link
Copy Markdown
Contributor

@annirudh annirudh commented Feb 6, 2021

https://wandb.atlassian.net/browse/WB-4519

Description

Adds the ability to delete files from the public API.

@codecov
Copy link
Copy Markdown

codecov Bot commented Feb 6, 2021

Codecov Report

Merging #1812 (692671b) into master (3de214b) will increase coverage by 0.02%.
The diff coverage is 93.33%.

Impacted file tree graph

@@            Coverage Diff             @@
##           master    #1812      +/-   ##
==========================================
+ Coverage   70.36%   70.39%   +0.02%     
==========================================
  Files         224      224              
  Lines       26529    26542      +13     
==========================================
+ Hits        18667    18684      +17     
+ Misses       7862     7858       -4     
Impacted Files Coverage Δ
tests/utils/mock_server.py 87.74% <50.00%> (-0.26%) ⬇️
tests/test_public_api.py 98.87% <100.00%> (+0.02%) ⬆️
wandb/apis/public.py 72.53% <100.00%> (+0.12%) ⬆️
wandb/sdk/lib/git.py 72.18% <0.00%> (ø)
wandb/sdk/internal/meta.py 85.90% <0.00%> (+3.35%) ⬆️

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 3de214b...692671b. Read the comment docs.

@raubitsj raubitsj changed the title Add ability to delete files from API [WB-4519] Add ability to delete files from API Feb 6, 2021
Copy link
Copy Markdown
Contributor

@vanpelt vanpelt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

@annirudh annirudh marked this pull request as ready for review February 8, 2021 17:18
@raubitsj raubitsj merged commit 7b7d09f into master Feb 8, 2021
@annirudh annirudh deleted the api/delete-file branch February 8, 2021 21:29
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.

3 participants