Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Deprecate ipfs file ls #5806

Open
dokterbob opened this issue Nov 29, 2018 · 3 comments
Open

Deprecate ipfs file ls #5806

dokterbob opened this issue Nov 29, 2018 · 3 comments
Labels
need/analysis Needs further analysis before proceeding topic/ls API Topic ls API

Comments

@dokterbob
Copy link
Contributor

Version information:

0.4.18

Type:

Enhancement

Description:

IPFS currently has the ipfs file ls, the ipfs files ls and the ipfs ls command, which is horribly confusing. Let's at least drop ipfs file ls already!

I would suggest explicitly adding a deprecation warning in the next release and to drop it entirely the one after. Note that there is also still #5805, which I think should be dealt with before removing this.

Related: #4580 and #3081
@kevina kevina added the topic/ls API Topic ls API label Dec 2, 2018
@Stebalien
Copy link
Member

Yeah, we should totally do this. It has been hidden for ages.

@Stebalien Stebalien added the help wanted Seeking public contribution on this issue label Dec 20, 2018
@magik6k
Copy link
Member

magik6k commented Jan 2, 2019

Also related to #5497

@ntninja ntninja mentioned this issue Jan 23, 2019
Closed
@Stebalien Stebalien added needs refinement and removed help wanted Seeking public contribution on this issue labels Apr 29, 2019
@Stebalien
Copy link
Member

Switching this to "needs refinement" as we need to think through:

  1. The precise API.
  2. Backwards compatibility.

@ec1oud ec1oud mentioned this issue Jun 11, 2019
Open
@hsanjuan hsanjuan added need/analysis Needs further analysis before proceeding and removed needs refinement labels Apr 16, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
need/analysis Needs further analysis before proceeding topic/ls API Topic ls API
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@dokterbob @Stebalien @hsanjuan @kevina @magik6k