-
Notifications
You must be signed in to change notification settings - Fork 91
Update tooling and protocols #15
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
Conversation
Not sure why the check is failing, compiles fine here. |
It looks like it's related to the .NET Core tooling. I'll try to have a look at it later this week - ping me if you do not hear back from me. |
Weird, I didn't change anything to the project itself. |
… of having to instantiate it
Some of these changes are breaking but useful. If you'd like to keep backwards-compatibility, you're welcome to modify them - obvious, I know :) |
@georgiosd There are some merge conflicts because the code which supports optional attributes has been merged. Could you rebase this against master and update the PR? You'll also need to re-generate the classes since optional attributes will now be marked properly. |
Sorry for the delay, will try to get to it this week |
Guys, apologies but I no longer have a need for this and very little time right now. |
@georgiosd Thanks for the heads up! I'll probably have a look at this in the future, no promises though :) |
No description provided.