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
Windows: CLI Improvement (TP3) #14838
Conversation
The Ansi parser and their associated actions have been decoupled. Now parsing results in call backs to an interface which performs the appropriate actions depending on the environment. This improvement provides a functional Vi experience and the vttest no longer panics. This PR replaces docker/docker moby#13224 with the latest console updates. Signed-off-by: John Howard <jhoward@microsoft.com>
This PR is needed for the TP3 build for correct ANSI support in Windows containers. But also improves the Linux experience significantly. |
@jhowardmsft just for my info; is TP3 to be presented at the same time as Docker 1.8? Wondering if these should be marked somehow to make sure they get in in time (if it's any of "our" worries here 😉) |
The current plan is that the docker daemon Windows release to support TP3 will be in 1.9.0-dev. I though we were supposed to have snapped to 1.8 earlier this week and the version bumped. I haven't seen that happen though? Certainly though, there are a few PRs which are still needed to support TP3, but we're very close now :) |
With the version bump, you mean the version bump here? https://github.com/docker/docker/blob/master/VERSION (it's 1.8-dev now) |
Yup, that's the one. |
LGTM |
LGTM |
merging, its a big one but if anything goes wrong we know where to find you ;) |
Windows: CLI Improvement (TP3)
Yup. on Bing searching for Nyancats 😉🐱🐱 |
Signed-off-by: John Howard jhoward@microsoft.com
@swernli @brendandixon @ahmetalpbalkan @jstarks @tiborvass @icecrime
The Ansi parser and their associated actions have been decoupled. Now
parsing results in call backs to an interface which performs the
appropriate actions depending on the environment.
This improvement provides a functional Vi experience and the vttest no
longer panics.
This PR replaces the stale docker/docker #13224.