Skip to content
This repository has been archived by the owner on Jun 26, 2023. It is now read-only.

fix: not remove file by mistakes #73

Merged
merged 2 commits into from
Mar 26, 2019
Merged

fix: not remove file by mistakes #73

merged 2 commits into from
Mar 26, 2019

Conversation

qiwaa
Copy link
Contributor

@qiwaa qiwaa commented Mar 26, 2019

@qiwaa qiwaa requested a review from schomatis as a code owner March 26, 2019 06:20
magik6k
magik6k reviewed Mar 26, 2019
View reviewed changes
ops.go Outdated Show resolved Hide resolved
@Stebalien
Copy link
Member

That's a nasty bug, thanks for catching it! Mind adding a test to test/sharness/t0250-files-api.sh?

@magik6k @qiwaa
Update ops.go
9fe9f96 
Co-Authored-By: chenminjian <727180553@qq.com>
@Stebalien Stebalien merged commit 30aee5f into ipfs:master Mar 26, 2019
@qiwaa
Copy link
Contributor Author

qiwaa commented Mar 26, 2019

OK, I would like to do that.

@Stebalien
Copy link
Member

Thanks!

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

@magik6k magik6k magik6k left review comments

@Stebalien Stebalien Stebalien approved these changes

@schomatis schomatis Awaiting requested review from schomatis

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@qiwaa @Stebalien @magik6k