Skip to content
This repository has been archived by the owner on Feb 12, 2024. It is now read-only.

fix: stats tests #1226

Merged
merged 2 commits into from
Feb 17, 2018
Merged

fix: stats tests #1226

merged 2 commits into from
Feb 17, 2018

Conversation

hacdias
Copy link
Member

@hacdias hacdias commented Feb 17, 2018

Fix #1225

@ghost ghost assigned hacdias Feb 17, 2018
@ghost ghost added the status/in-progress In progress label Feb 17, 2018
@hacdias
Copy link
Member Author

hacdias commented Feb 17, 2018

@diasdavid CircleCI passed on all tests 😄

@victorb
Copy link
Member

victorb commented Feb 17, 2018

Tests are currently failing master and this looks fine, thanks!

@victorb victorb merged commit 698b5b6 into master Feb 17, 2018
@ghost ghost removed the status/in-progress In progress label Feb 17, 2018
@hacdias hacdias deleted the fix-stats-tests branch February 17, 2018 11:42
@daviddias
Copy link
Member

thank you @hacdias :)

MicrowaveDev pushed a commit to galtproject/js-ipfs that referenced this pull request May 22, 2020
BREAKING CHANGE:

Where `PeerID`s were previously [CID]s, now they are Strings

- `ipfs.bitswap.stat().peers[n]` is now a String (was a CID)
- `ipfs.dht.findPeer().id` is now a String (was a CID)
- `ipfs.dht.findProvs()[n].id` is now a String (was a CID)
- `ipfs.dht.provide()[n].id` is now a String (was a CID)
- `ipfs.dht.put()[n].id` is now a String (was a CID)
- `ipfs.dht.query()[n].id` is now a String (was a CID)
- `ipfs.id().id` is now a String (was a CID)
- `ipfs.id().addresses[n]` are now [Multiaddr]s (were Strings)
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.

3 participants