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.

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

fix ls command to use the new coreinterface types #6051

Merged
merged 3 commits into from
Mar 7, 2019
Merged

Conversation

Stebalien
Copy link
Member

@Stebalien Stebalien requested a review from Kubuxu as a code owner March 5, 2019 04:30
@ghost ghost assigned Stebalien Mar 5, 2019
@ghost ghost added the status/in-progress In progress label Mar 5, 2019
@Stebalien Stebalien added the status/blocked Unable to be worked further until needs are met label Mar 5, 2019
@Stebalien Stebalien requested a review from magik6k March 5, 2019 04:30
Copy link
Member

@magik6k magik6k left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

1 Nit, overall looks good

case cid.Raw:
// No need to check with raw leaves
lnk.Type = coreiface.TFile
lnk.Size = lnk.Link.Size
lnk.Size = linkres.Link.Size
case cid.DagProtobuf:
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This should also fill lnk.Target for symlinks.

@Kubuxu
Copy link
Member

Kubuxu commented Mar 5, 2019

Sharness is not happy.

@Stebalien Stebalien force-pushed the feat/nicer-ls branch 5 times, most recently from c38362a to f39b9a4 Compare March 7, 2019 00:52
License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
See: ipfs/interface-go-ipfs-core#14

License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
License: MIT
Signed-off-by: Steven Allen <steven@stebalien.com>
@Stebalien Stebalien removed the status/blocked Unable to be worked further until needs are met label Mar 7, 2019
@Stebalien Stebalien merged commit 245c40b into master Mar 7, 2019
@ghost ghost removed the status/in-progress In progress label Mar 7, 2019
@hacdias hacdias deleted the feat/nicer-ls branch May 9, 2023 10:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants