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

Rebase IPFS to v0.4.19 #1528

Merged
merged 14 commits into from
Apr 3, 2019
Merged

Rebase IPFS to v0.4.19 #1528

merged 14 commits into from
Apr 3, 2019

Conversation

cpacia
Copy link
Member

@cpacia cpacia commented Mar 28, 2019

No description provided.

@coveralls
Copy link

coveralls commented Apr 1, 2019

Coverage Status

Coverage decreased (-0.2%) to 35.561% when pulling 66e6e2e on ipfsrebase into 80b46e7 on master.

Copy link
Member

@placer14 placer14 left a comment

Choose a reason for hiding this comment

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

Two questions. Overall it seems GTM. Thanks @cpacia 🤝

@@ -9,7 +9,7 @@ import (
"github.com/tyler-smith/go-bip39"
)

var keyHex = "08011260499228645d120d15b5008b1da0b9dba898df328001ea03c0be84a64c41d205ff1b8339a303cd8cf2945b66c89ac29fa90e79731d67000694284791af404eeb1f1b8339a303cd8cf2945b66c89ac29fa90e79731d67000694284791af404eeb1f"
Copy link
Member

Choose a reason for hiding this comment

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

Why did this change?

Copy link
Member Author

Choose a reason for hiding this comment

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

They changed the protobuf serialization of the key. It broke our test. Shouldn't break the functionality though.

Copy link
Member

Choose a reason for hiding this comment

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

We don't use that protobuf serialization anywhere for longterm storage, right?

schema/manager.go Outdated Show resolved Hide resolved
@cpacia cpacia merged commit b663812 into master Apr 3, 2019
@cpacia cpacia deleted the ipfsrebase branch April 5, 2019 15:11
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants