Skip to content
This repository has been archived by the owner on Mar 10, 2020. It is now read-only.

test: make files.ls results ordering consistent #342

Merged
merged 1 commit into from
Aug 2, 2018

Conversation

achingbrain
Copy link
Collaborator

@achingbrain achingbrain commented Aug 2, 2018

Previously we'd get files in different orders between files.ls() and files.ls({long: true}) due to a bug in go-ipfs (see ipfs/kubo#5181)

Since this has been resolved we can now expect consistent ordering.

@ghost ghost assigned achingbrain Aug 2, 2018
@ghost ghost added the in progress label Aug 2, 2018
@achingbrain achingbrain requested review from alanshaw and removed request for travisperson and victorb August 2, 2018 08:56
Previously we'd get files in different orders between `files.ls()` and
`files.ls({long: true})` due to a bug in `go-ipfs` (see ipfs/kubo#5181)

Since this has been resolved we can now expect consistent ordering.
@achingbrain achingbrain changed the title test: now ipfs/go-ipfs#5181 has been resolved we can expect consistent ordering make files.ls results ordering consistent Aug 2, 2018
@achingbrain achingbrain changed the title make files.ls results ordering consistent test: make files.ls results ordering consistent Aug 2, 2018
@achingbrain
Copy link
Collaborator Author

achingbrain commented Aug 2, 2018

This will break the js-ipfs tests until ipfs-inactive/js-ipfs-mfs#6 is merged & released to restore compatibility on the js side. I will sort that out as soon as this is merged.

@alanshaw alanshaw merged commit 3f7a775 into master Aug 2, 2018
@ghost ghost removed the in progress label Aug 2, 2018
@alanshaw alanshaw deleted the consistent-ls-list-order branch August 2, 2018 14:37
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants