-
-
Notifications
You must be signed in to change notification settings - Fork 8.7k
Open
Labels
A-needs-triagingA Selenium member will evaluate this soon!A Selenium member will evaluate this soon!C-dotnet.NET Bindings.NET BindingsI-enhancementSomething could be betterSomething could be better
Description
Description
Nobody knows when Async #14067 will be in place, and moreover how to do it smoothly without pain for end-users.
I propose to make small improvement now, even before v5.
Current
var driver = new ChromeDriver();Proposal:
var driver = await ChromeDriver.StartAsync();Have you considered any alternatives or workarounds?
In any case constructor cannot be async, we will rework it entirely. Ha, and it means everybody has to use another API.
I propose alternative approach here.
Bad?
var driver = new ChromeDriver();
await driver.StartAsync();Better:
var driver = await ChromeDriver.StartAsync();We can try to make it without breaking changes. Seems it is small step, but it contributes significantly. This is most tricky part of migration.
Reactions are currently unavailable
Metadata
Metadata
Assignees
Labels
A-needs-triagingA Selenium member will evaluate this soon!A Selenium member will evaluate this soon!C-dotnet.NET Bindings.NET BindingsI-enhancementSomething could be betterSomething could be better