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

[🚀 Feature]: [dotnet] [bidi] Introduce EmptyResult as a type #15562

Open
nvborisenko opened this issue Apr 3, 2025 · 0 comments
Open

[🚀 Feature]: [dotnet] [bidi] Introduce EmptyResult as a type #15562

nvborisenko opened this issue Apr 3, 2025 · 0 comments
Labels
B-devtools Includes everything BiDi or Chrome DevTools related C-dotnet .NET Bindings I-enhancement Something could be better

Comments

@nvborisenko
Copy link
Member

nvborisenko commented Apr 3, 2025

Description

Now:

public Task ActivateAsync(ActivateOptions? options = null)

Proposal:

public Task<EmptyResult> ActivateAsync(ActivateOptions? options = null)

Why:
Because EmptyResult is not empty :)

EmptyResult = {
  Extensible
}

Have you considered any alternatives or workarounds?

No response

@nvborisenko nvborisenko added the I-enhancement Something could be better label Apr 3, 2025
@github-actions github-actions bot added B-devtools Includes everything BiDi or Chrome DevTools related C-dotnet .NET Bindings labels Apr 3, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
B-devtools Includes everything BiDi or Chrome DevTools related C-dotnet .NET Bindings I-enhancement Something could be better
Projects
None yet
Development

No branches or pull requests

1 participant