Skip to content
This repository has been archived by the owner on Aug 2, 2021. It is now read-only.

remove api.Put #1350

Closed
acud opened this issue Apr 23, 2019 · 0 comments
Closed

remove api.Put #1350

acud opened this issue Apr 23, 2019 · 0 comments
Labels
cleanup code completion, add comments and more good first issue

Comments

@acud
Copy link
Member

acud commented Apr 23, 2019

the method is not used anywhere but tests and is marked as deprecated since a couple of years. hence, we should remove it. the functionality should be pulled out to testutil. it is basically just a wrapper that creates a manifest and uploads a string content to the file store.
@acud acud added good first issue cleanup code completion, add comments and more labels Apr 23, 2019
@acud acud mentioned this issue May 3, 2019
Merged
@acud acud closed this as completed May 5, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
cleanup code completion, add comments and more good first issue
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@acud