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

Target netstandard2.0 and use the new csproj format. #41

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

Conversation

rwb196884
Copy link

Add endpoints listEvents and listRunnerBook.

Use keep alive.

RescriptClient seems to be broken.

Add endpoints listEvents and listRunnerBook.

Use keep alive.

RescriptClient seems to be broken.
<ImplicitUsings>disable</ImplicitUsings>
<Nullable>disable</Nullable>
<LangVersion>8</LangVersion>
<VersionPrefix>1.0.1</VersionPrefix>
Copy link

Choose a reason for hiding this comment

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

This is going to be a breaking change for some people - should it be at least a minor version revision, if not major?

@rwb196884
Copy link
Author

The thing is pretty broken already!

Happy to tody it up a bit; it was mainly for my own use but I thought I'd create a merge request on the off-chance that it coulld help anyone else.
How do you want to proceed?

I would suggest.

  • Increment the major version number.
  • Remove RescriptClient
  • Restore the original constructor to JsonRpcClient and have the new methods that use the new URL endpoints added in the new constructor throw a NotImplementedException if they are called.

@rwb196884
Copy link
Author

Hello?

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