Skip to content
This repository was archived by the owner on Mar 5, 2025. It is now read-only.

feat: bring in ethereum-bloom-filters#3137

Merged
nivida merged 7 commits intoweb3:1.xfrom
joshstevens19:feat/use-blooms-lib
Oct 17, 2019
Merged

feat: bring in ethereum-bloom-filters#3137
nivida merged 7 commits intoweb3:1.xfrom
joshstevens19:feat/use-blooms-lib

Conversation

@joshstevens19
Copy link
Contributor

@joshstevens19 joshstevens19 commented Oct 16, 2019

  • blooms in web3 were never exposed and had a few bugs in them. The package has fixed them all and fully 100% tested
  • blooms is not used enough client side and this library is a single point of truth for related bloom stuff
  • will go into web3 and etherjs so they share the same codebase

@nivida the package itself has all the docs so Sam said he will extend the web3 docs to reference them, he knows how he wants that showing so leaving that part to him.

@joshstevens19 joshstevens19 requested a review from nivida October 16, 2019 16:01
@coveralls
Copy link

coveralls commented Oct 16, 2019

Coverage Status

Coverage decreased (-0.09%) to 84.78% when pulling 3d74bd3 on joshstevens19:feat/use-blooms-lib into 400d24b on ethereum:1.x.

@nivida nivida added 1.x 1.0 related issues Feature Request In Progress Currently being worked on labels Oct 17, 2019
@nivida nivida removed the In Progress Currently being worked on label Oct 17, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

1.x 1.0 related issues Feature Request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants