Skip to content
This repository has been archived by the owner on Jul 15, 2018. It is now read-only.

Release/v0.10.0 #144

Open
wants to merge 39 commits into
base: master
Choose a base branch
from
Open

Release/v0.10.0 #144

wants to merge 39 commits into from

Conversation

ebuchman
Copy link
Contributor

@ebuchman ebuchman commented Jul 1, 2018

When looking into the version of go-crypto that got merged into Tendermint to try and accurately document the breaking changes, I noticed a few things:

  • Changelog for v0.8.0 on master was missing some entries
  • Changelog for v0.8.0 was missing altogether on develop
  • develop was 37 commits ahead, 6 commits behind master

It looks like develop is what got merged into tendermint, but it has more changes that aren't captured on master or in a release, so we can't actually say "version X of go-crypto was merged into tendermint". This is somewhat unfortunate.

As for the commits on master not on develop, I'm not exactly sure what happened, but some seem to be in Tendermint and some don't - in particular the recent This is a combination of X commits. commits are missing.

I have done the following:

  • Update the changelog for v0.8.0 on master
  • Merge master back into develop (there were some conflicts...)
  • Prepare release v0.10.0, the final release for go-crypto

I'm not sure if there is something here that is missing in the tendermint/crypto or in the keys that got moved to SDK.

I'm mostly just concerned with accurately documenting the breaking changes in go-crypto so that when we release Tendermint with crypto moved in, we can say exactly what happened.

cwgoes and others added 30 commits May 14, 2018 15:26
* Return errors on priv.Sign(), priv.PubKey()

* Add CreateLedger, CreateOffline

* Add switch on .Sign() for Ledger wallets

* Add offline signing switch on .Sign()

* Use MustUnmarshalBinaryBare()

* Add confirmation to delete offline/Ledger keys

* Lowercase error message

* Add human-readable .GetType() function to Info interface

* Rename CryptoAlgo => SignAlgo

* assert.Nil(t, err) => assert.NoError(t, err)
* move from tmlibs 213
* expose KVPair, simpleproofsfrommap returns keys
make PubKeyEd25519.Address() returns the first 20 bytes of the hash of the raw 32-byte pubkey, no amino required
include an example, do not err check on info ... bur for empty info
instead
…name

# Conflicts:
#	Gopkg.lock
#	keys/keybase_test.go
#	signature_test.go
version bump
* Remove error from priv.PubKey()

* Update changelog
* circleci 2.0
liamsi and others added 8 commits June 15, 2018 15:17
@ebuchman ebuchman requested a review from jaekwon as a code owner July 1, 2018 04:26
@liamsi
Copy link
Contributor

liamsi commented Jul 1, 2018

It looks like develop is what got merged into tendermint, but it has more changes that aren't captured on master or in a release, so we can't actually say "version X of go-crypto was merged into tendermint". This is somewhat unfortunate.

Yes, develop was merged into tendermint. We thought, we might "retire" go-crypto by making it a private repo as as soon as the merge is through. Hence we took the develop branch and did not create a release before (cc @cwgoes)

I'm not sure if there is something here that is missing in the tendermint/crypto or in the keys that got moved to SDK.

We deleted the entire keys package here (7ebc7c0 and a589e0e). It is not needed in tendermint. Merging into cosmos-sdk happend exactly like into to tendermint (from develop) but with reverting these two commits (and deleting everything else).

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.

6 participants