Skip to content

fix: report addresses without checksum#384

Merged
Theodus merged 2 commits intomainfrom
release
Aug 28, 2023
Merged

fix: report addresses without checksum#384
Theodus merged 2 commits intomainfrom
release

Conversation

@Theodus
Copy link
Member

@Theodus Theodus commented Aug 28, 2023

The checksum serialization is causing issues for data science pipelines. Luckily the new Address type uses lowercase hex for debug formatting.

@Theodus Theodus requested a review from LNSD August 28, 2023 17:29
Copy link
Contributor

@LNSD LNSD left a comment

Choose a reason for hiding this comment

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

LGTM ✅

@Theodus Theodus merged commit 2bb3d22 into main Aug 28, 2023
@Theodus Theodus deleted the release branch August 28, 2023 17:44
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants