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
Add event filtering patterns to stream event type #10285
Add event filtering patterns to stream event type #10285
Conversation
Codecov Report
@@ Coverage Diff @@
## master #10285 +/- ##
=======================================
Coverage 85.39% 85.39%
=======================================
Files 339 339
Lines 13854 13857 +3
=======================================
+ Hits 11830 11833 +3
Misses 2024 2024
Continue to review full report at Codecov.
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks good in general, I have only very minor suggestions 👍 Also, please see the suggestion from @medikoo in linked issue, it might impact this PR a little bit.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks great in my opinion 👍 @medikoo could you also take a look before merging?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
That looks great @fredericbarthelet
Great thanks for fast action on that!
Addresses: #10284