Release/v0.12.0 #216
Release/v0.12.0 #216
Conversation
* DecodeInt{8, 16} negative limit checks + tests
Updates #120
Also adds a test for Byteslice encoding and decoding
roundtripping and part slicing then rejoining.
* fix circleci2.0
* some metalinter issues
* skip over empty structs by default * slightly more info on panics
* update changelog for release * prep release 0.11.0
- fix decoding of skipped fields in time, or completely skipped time
- neccessary because sec=0 and ns=0 do not result in time.Time{} and vice versa
* update changelog for release
- top-level entry functions called with BinFieldNum:1 to properly encode e.g. arrays of structs (see non-time related test) - add defaultValue method, different from #196 it deals with multiply nested pointers
Removes dependency on tmlibs/common
- compare to proto3 behaviour - test for pointers to empty struct and nil pointer Signed-off-by: Liamsi <[email protected]>
- use []byte{} not []byte(nil)
Signed-off-by: Liamsi <[email protected]>
This reverts commit e90abf1
…writeemptyptr_tests
Write empty (non-nil) struct pointers, unless (is list element and empty_elements isn't set)
# Conflicts: # CHANGELOG.md
|
LGTM! |
|
Thanks Jae. I think I did. But I'm actually not sure if I did only run the binary fuzzer. Let me rerun both.
That is a good idea. Are there any templates for releases? Similar to PR templates with checklists? Let me see. |
|
OK, just to be sure, I reran all three: |
Merge master into develop
This release fixes a bunch of edge cases since the initial change to proto3 (default values on time encoding, empty structs, nil pointer to empty structs). It also fixes the circleci 1.0 -> 2.0 migration on master and removes the dependency on tmlibs/common.
Full change log:
https://github.com/tendermint/go-amino/blob/b501f6cbae3af423a404e13233267656e1b75a66/CHANGELOG.md
It makes sense to merge #218 first and rebase on this. Apparently, we didn't merge master back into develop last time we did a release.