-
Notifications
You must be signed in to change notification settings - Fork 21
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
Replace messageHandler with wrpHandler #121
Conversation
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.
Good job!
Codecov Report
@@ Coverage Diff @@
## master #121 +/- ##
==========================================
+ Coverage 52.35% 56.13% +3.77%
==========================================
Files 9 10 +1
Lines 680 734 +54
==========================================
+ Hits 356 412 +56
+ Misses 317 314 -3
- Partials 7 8 +1
Continue to review full report at Codecov.
|
go.mod
Outdated
@@ -17,7 +17,7 @@ require ( | |||
github.com/spf13/viper v1.6.1 | |||
github.com/stretchr/testify v1.3.0 | |||
github.com/xmidt-org/webpa-common v1.6.3 | |||
github.com/xmidt-org/wrp-go/v2 v2.0.1 | |||
github.com/xmidt-org/wrp-go/v2 v2.0.2-0.20200331181148-cbb4fba7a6e0 |
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.
will update soon - likely with v3
This change replaces the existing WRP Message handler with the existing WRP handler. All unit tests were brought from the message handler code base.