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
Implement deleteFile graphql mutation (#5292). #7937
Implement deleteFile graphql mutation (#5292). #7937
Conversation
Codecov Report
@@ Coverage Diff @@
## master #7937 +/- ##
==========================================
- Coverage 32.96% 32.95% -0.02%
==========================================
Files 1197 1197
Lines 13020 13027 +7
Branches 1286 1286
==========================================
+ Hits 4292 4293 +1
- Misses 7885 7891 +6
Partials 843 843
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
c9c949f
to
f7e9833
Compare
Hi @francois2metz just a small update before we can merge ;) |
@alexandrebodin I'll try to do it today. Thanks for the reminder ;). |
e521ea5
to
21dde85
Compare
Signed-off-by: François de Metz <francois@2metz.fr>
Signed-off-by: François de Metz <francois@2metz.fr>
21dde85
to
bec9595
Compare
Signed-off-by: François de Metz <francois@2metz.fr>
bec9595
to
88fd2a9
Compare
I updated the PR to check if the file exists, and added a commit to fix a typo. I can remove it from this branch if you don't want to mix the history. |
Thansk for this PR ! |
This fix #5292 by implementing and testing the removeFile mutation.