-
Notifications
You must be signed in to change notification settings - Fork 12
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
WRP source #144
WRP source #144
Conversation
translation/service_test.go
Outdated
@@ -24,7 +24,7 @@ func TestSendWRP(t *testing.T) { | |||
|
|||
var expected = wrp.MustEncode(wrp.Message{ | |||
Type: wrp.SimpleRequestResponseMessageType, | |||
Source: "local/test", | |||
Source: "local", |
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.
can we change this to dns:localhost
to make it a bit more realistic example/test? You might have to update some other part of the tests
Codecov Report
@@ Coverage Diff @@
## master #144 +/- ##
==========================================
- Coverage 79.5% 79.45% -0.06%
==========================================
Files 11 11
Lines 405 404 -1
==========================================
- Hits 322 321 -1
Misses 82 82
Partials 1 1
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 great! Just one small change.
CHANGELOG.md
Outdated
@@ -5,6 +5,8 @@ The format is based on [Keep a Changelog](http://keepachangelog.com/en/1.0.0/) | |||
and this project adheres to [Semantic Versioning](http://semver.org/spec/v2.0.0.html). | |||
|
|||
## [Unreleased] | |||
- moving partnerIDs to tr1d1um |
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.
This is part of the v0.2.1 release (below), so shouldn't be here anymore.
fixes
[https://github.com//issues/142]