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

Spectre #362

Closed
wants to merge 11 commits into from
Closed

Spectre #362

wants to merge 11 commits into from

Conversation

drewburlingame
Copy link
Collaborator

@drewburlingame drewburlingame commented Oct 26, 2021

bring the goodness of the Spectre AnsiConsoele goodness readily available in CommandDotNet. Two great tastes that go great together https://www.youtube.com/watch?v=DJLDF6qZUX0

Spectre has a much better set of features for writing complex output and prompting, and they are problems I don't feel we need to solve in CommandDotNet.

This PR Includes

  • IAnsiConsole parameter resolver to inject into command methods
  • a test package so the output to IAnsiConsole is passed through the TestTools. Spectre appears to be following some similar test patterns which made this easier. There are a few caveats, but I think they are acceptable.

Still needs documentation, and a design review from the Spectre crew if they're willing to oblige.

needed to use Spectre's ReadKeyAsyc to replicate
System.Console behavior
@drewburlingame drewburlingame deleted the spectre branch November 3, 2021 22:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant