Skip to content

Conversation

cmodijk
Copy link
Member

@cmodijk cmodijk commented May 2, 2017

Copy link

@Nyholm Nyholm left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't we need more changes than the docs?

@cmodijk
Copy link
Member Author

cmodijk commented Jun 12, 2017

@Nyholm No I checked the MessageInEnvelopSerializer is no where to be found.

@SimpleBus SimpleBus deleted a comment from coveralls Jun 12, 2017
@codecov
Copy link

codecov bot commented Jun 12, 2017

Codecov Report

Merging #6 into master will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master       #6   +/-   ##
=======================================
  Coverage   78.26%   78.26%           
=======================================
  Files           2        2           
  Lines          23       23           
=======================================
  Hits           18       18           
  Misses          5        5

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 40bdaee...ef22ede. Read the comment docs.

@cmodijk cmodijk self-assigned this Jun 14, 2017
@cmodijk cmodijk merged commit 3b223cf into master Aug 28, 2017
@cmodijk cmodijk deleted the fix-4 branch August 28, 2017 06:54
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.

2 participants