Skip to content

Conversation

@martinmaillard
Copy link

Currently, the old file is only deleted if no new file is provided.

@coveralls
Copy link

Coverage Status

Coverage remained the same when pulling d244446 on martinmaillard:remove-old-file-when-updating into 14e2911 on codingjoe:master.

@codingjoe
Copy link
Owner

@martinmaillard thanks for the contribution
would you mind adding a test for this bug?

@codingjoe
Copy link
Owner

@martinmaillard I wrote the test myself and merged. Thanks for your contribution.

@martinmaillard
Copy link
Author

Oops, sorry, I totally forgot. Thank you!

@codingjoe
Copy link
Owner

you're welcome, thanks for your support. I hope you like the library.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants