Skip to content

Vdb 1424 remove hashed addresses#121

Merged
yaoandrew merged 6 commits intostagingfrom
vdb-1424-remove-hashed-addresses
Aug 26, 2020
Merged

Vdb 1424 remove hashed addresses#121
yaoandrew merged 6 commits intostagingfrom
vdb-1424-remove-hashed-addresses

Conversation

@yaoandrew
Copy link
Copy Markdown

No description provided.

@yaoandrew yaoandrew force-pushed the vdb-1424-remove-hashed-addresses branch 2 times, most recently from bbf2758 to c1bf259 Compare August 11, 2020 18:09
Copy link
Copy Markdown

@rmulhol rmulhol left a comment

Choose a reason for hiding this comment

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

Looking good! Will be curious to take one more look after the rebase

github.com/fjl/memsize v0.0.0-20190710130421-bcb5799ab5e5 // indirect
github.com/gballet/go-libpcsclite v0.0.0-20191108122812-4678299bea08 // indirect
github.com/gorilla/websocket v1.4.1 // indirect
github.com/ethereum/go-ethereum v1.9.8
Copy link
Copy Markdown

Choose a reason for hiding this comment

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

If we're bumping this, might be cool to bump it even higher - but idk if it matters given the replace below 🤔

Copy link
Copy Markdown

@elizabethengelman elizabethengelman left a comment

Choose a reason for hiding this comment

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

Looking good!

I'm wondering if some associated changes need to be made to the transformers repo as well - like in configured transformer component tests.

@yaoandrew yaoandrew force-pushed the vdb-1424-remove-hashed-addresses branch from 781b86f to 26792e7 Compare August 26, 2020 18:53
@yaoandrew yaoandrew merged commit 1efa92c into staging Aug 26, 2020
@yaoandrew yaoandrew deleted the vdb-1424-remove-hashed-addresses branch August 26, 2020 19:06
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.

3 participants