Skip to content
This repository was archived by the owner on Dec 26, 2017. It is now read-only.

Use PlainMessage instead of the deprecated DefaultMessage #5

Merged
merged 1 commit into from
Dec 27, 2016
Merged

Use PlainMessage instead of the deprecated DefaultMessage #5

merged 1 commit into from
Dec 27, 2016

Conversation

holtkamp
Copy link
Contributor

As suggested in #4

@coveralls
Copy link

coveralls commented Dec 26, 2016 •
edited
Loading

Coverage Status

Coverage remained the same at 98.502% when pulling 7437511 on holtkamp:patch-use-PlainMessage into 7c24ee5 on SimpleBus:master.

@ruudk ruudk merged commit 4718a41 into SimpleBus:master Dec 27, 2016
@ruudk
Copy link
Contributor

ruudk commented Dec 27, 2016

Thanks!

@holtkamp holtkamp deleted the patch-use-PlainMessage branch December 27, 2016 20:22
@holtkamp
Copy link
Contributor Author

@ruudk thanks! Does this justify tagging/releasing a new minor version 1.1.0? Currently I need to use dev-master in my composer.json which should probably be considered a bad practice 😊

@holtkamp
Copy link
Contributor Author

shameless bump on this?

@lakiboy
Copy link
Contributor

lakiboy commented Jan 31, 2017

@holtkamp Done

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Reviewers
1 more reviewer

@ruudk ruudk ruudk approved these changes

Reviewers whose approvals may not affect merge requirements
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
4 participants
@holtkamp @coveralls @ruudk @lakiboy