Add support for encoding time.Duration #19
Merged
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Manually cherry-picked changes from upstream repo: 77976d5b6d9c014b96d4be0b169d0c4dc9ac1a0b 5467d2f6f6238b93ec0a064bada8dcf9f081e603 618077c9521bd18d4c5b22dcb45444c2d8ffa3e9 66ba8975dc3b0a050050e60694dd446a7e64431c fd0c04e0f5ca1995ec6b166faa8f2f30ec0b1c99 222902d7cb87b1061282d75abec3327692e549e8
zeldovich
approved these changes
Oct 11, 2022
algorandskiy
approved these changes
Oct 12, 2022
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Support for
time.Durationis necessary in order to serialize agreement crash state using msgp instead of reflect for algorand/go-algorand#4550These changes are manually cherry-picked changes from the upstream tinylib repo commits listed below. They were done manually because our repos have diverged enough that real cherry-picks were introducing a lot of conflicts. They are merged into
masterbut not yet included in a tagged release.77976d5b6d9c014b96d4be0b169d0c4dc9ac1a0b
5467d2f6f6238b93ec0a064bada8dcf9f081e603
618077c9521bd18d4c5b22dcb45444c2d8ffa3e9
66ba8975dc3b0a050050e60694dd446a7e64431c
fd0c04e0f5ca1995ec6b166faa8f2f30ec0b1c99
222902d7cb87b1061282d75abec3327692e549e8