Skip to content
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

Added continuous integration #2

Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

Conversation

yallie
Copy link

@yallie yallie commented Sep 23, 2017

Hello,

I've taken the liberty to update all referenced Nuget packages to their latest versions and add the continuous integration settings to the project (both Travis-CI and Appveyor). Also, I've made these minor changes:

  • Removed numerous Send method overloads from IClient and IHost interfaces
  • Replaced the overloads with the equivalent extension methods to keep the same API
  • Added copyright headers to the interfaces, same as with other files
  • Removed extra src folder (src/src/MessageWire.sln -> src/MessageWire.sln)
  • Added .editorconfig file so that Visual Studio 2017 picks up the project's tab settings automatically
  • Added badges to the README to display the current build status and the package version like this:

Appveyor build status Travis-CI build status MessageWire NuGet version

For some reason Github UI doesn't display all the file renames properly (although Git itself detects them just fine). I'm a bit disappointed that the diff looks like I've made a lot of changes — there are not that many changes, actually.

Last, but not least — thanks for this project! Hope it's not abandoned! :)

Regards, Alex

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.

1 participant