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

Sdk/c sharp #111

Merged
merged 18 commits into from
Jul 20, 2020
Merged

Sdk/c sharp #111

merged 18 commits into from
Jul 20, 2020

Conversation

rvowles
Copy link
Contributor

@rvowles rvowles commented Jul 20, 2020

Description

Full C# SDK plus changes discovered when working on it. Not ready for merge but:

  • needs logging
  • needs some unit testing around event source
  • need to check if properly deals with server not even being there (failed)

Fixes #67

Type of change

Please delete options that are not relevant.

  • New feature (non-breaking change which adds functionality)

How Has This Been Tested?

Unit tests + e2e tests. I have also built a nuget package for it and installed that in a separate console app and run it up.

@IrinaSouth IrinaSouth merged commit c072ba2 into master Jul 20, 2020
@rvowles rvowles deleted the sdk/c-sharp branch July 26, 2020 09:36
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.

C# / .Net SDK
2 participants