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.

Sign up for GitHub

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

Jump to bottom

Change RestServer.ThreadSafeStop to Extension Method on IRestServer #151

Closed
scottoffen opened this issue Nov 17, 2016 · 0 comments
Closed

Change RestServer.ThreadSafeStop to Extension Method on IRestServer #151

scottoffen opened this issue Nov 17, 2016 · 0 comments
Assignees
@scottoffen
Labels
enhancement
Milestone
4.0.0.252

Comments

@scottoffen
Copy link
Owner

As the IRestServer interface does not contain the method ThreadSafeStop, but the RestServer implementation does, in order to ensure that the method - which is an incredibly simple implementation - is always available it will be moved to be an extension method on the interface.

Should probably check to see what other methods on other classes could be moved to extension methods for the same reason.
@scottoffen scottoffen added this to the 4.0.0.x milestone Nov 17, 2016
@scottoffen scottoffen assigned scottoffen and unassigned scottoffen Nov 17, 2016
@scottoffen scottoffen changed the title Move ThreadSafeStop to Extension Method Change RestServer.ThreadSafeStop to Extension Method on IRestServer Nov 18, 2016
scottoffen pushed a commit that referenced this issue Nov 18, 2016
Fix for #151
893bae2
@scottoffen scottoffen self-assigned this Dec 30, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@scottoffen scottoffen

Labels
enhancement
Projects
None yet
Milestone
4.0.0.252
Development

No branches or pull requests
1 participant
@scottoffen