Use Async name for async APIs

[dansiegel]

Description

Several API's within Shiny such as IJobManager.Schedule or IJobManager.Run look like they are synchronous when they actually operate asynchronously. This can lead to confusion where you might think you can initialize your jobs like:

manager.Schedule(someJob);
manager.Run("someJob");

However with them being async methods it is possible that the job may not be scheduled when Run is called. Adopting a Async suffix for async API's would be helpful to developers as it would be self evident that it's async.

await manager.ScheduleAsync(someJob);
await manager.Run("someJob");

Version of Library

• 1.0.0.2-beta

[close-issue-app]

This issue is closed because it does not meet our issue template. Please read it.

[aritchie]

Thanks for the suggestion Dan. I'm actually anti-postfix async on methods. The type defines the outcome in my opinion. I also mix between between observables & tasks throughout the project. I don't want to say "ThisObservable" and "ThisAsync" everywhere.

[lock]

This thread has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs.