Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

WIP Msg filter #872

Draft
wants to merge 7 commits into
base: development
Choose a base branch
from
Draft

WIP Msg filter #872

wants to merge 7 commits into from

Conversation

JakeHartnell
Copy link
Member

@JakeHartnell JakeHartnell commented Aug 2, 2024

Was feeling inspired on the plane, this is very much WIP and should be considered a strawman. Want to make sure we get the API and filtering right before we actually hook it up to a proposal module.

Feedback needed, hopefully the new cw-msg-filter README provides a good overview of current usage and API.

Hopefully Closes #694

Close this PR if someone makes a better version.

Copy link

codecov bot commented Aug 2, 2024

Codecov Report

Attention: Patch coverage is 92.23986% with 44 lines in your changes missing coverage. Please review.

Project coverage is 96.61%. Comparing base (75272b9) to head (83818f5).

Files Patch % Lines
packages/cw-msg-filter/src/lib.rs 92.23% 44 Missing ⚠️
Additional details and impacted files
@@               Coverage Diff               @@
##           development     #872      +/-   ##
===============================================
- Coverage        96.64%   96.61%   -0.04%     
===============================================
  Files              240      241       +1     
  Lines            65294    65862     +568     
===============================================
+ Hits             63106    63631     +525     
- Misses            2188     2231      +43     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

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.

Proposal Module Message Limits
1 participant