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

Allow ConsoleDriver to be customizable #115

Open
ryfu-msft opened this issue Apr 20, 2021 · 1 comment · May be fixed by #202
Open

Allow ConsoleDriver to be customizable #115

ryfu-msft opened this issue Apr 20, 2021 · 1 comment · May be fixed by #202
Labels
enhancement New feature or request
Milestone

Comments

@ryfu-msft
Copy link
Contributor

The current DefaultConsoleDriver implementation is internal and makes it impossible to override any of the behaviors of the DefaultConsoleDriver class to fit my needs. Some suggestions that I have would be:

  • Make the IConsoleDriver interface and DefaultConsoleDriver class public so that subclasses could inherit and override the method's behaviors.
  • Plumb through a way to override the ConsoleDriver property , so that we could specify our own custom implementation of a ConsoleDriver

I believe that these changes builds on such a neat tool and would make the tool more fun and flexible for users to play around with.

@ryfu-msft ryfu-msft changed the title Allow drivers to be customizable Allow ConsoleDriver to be customizable Apr 20, 2021
@shibayan shibayan added the enhancement New feature or request label Apr 21, 2021
@shibayan shibayan added this to the Backlog milestone Apr 21, 2021
@shibayan
Copy link
Owner

It is possible that it will be customizable in the future.

Since the implementation of ConsoleDriver is difficult considering the multiple language support and the interface is not completely solidified, I basically need your help to improve the default driver.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants