-
Notifications
You must be signed in to change notification settings - Fork 889
Command-line build log overwrites previous CLI output #1506
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
Comments
I've seen this... I'd like to have a better story for how this works, because it's a bit unfortunately janky right now. |
|
I think the thing in that PR is intentional, the select list gets removed and replaced with the selection |
Oh, right you are @deansheather, thanks for clearing that up! |
I've found one case where this is always reproducible:
In 2 |
This issue is becoming stale. In order to keep the tracker readable and actionable, I'm going close to this issue in 7 days if there isn't more activity. |
This still happens on occasion, but needs deeper investigation to narrow down cause and where it happens (for instance, it has also happened when prompting for terraform variable input which is confusing since part or all of the message is missing -- this may or may not have been fixed). |
This issue is becoming stale. In order to keep the tracker readable and actionable, I'm going close to this issue in 7 days if there isn't more activity. |
@kylecarbs is this fixed by #4740? |
yup |
In some cases, the CLI can overwrite previous output.
Steps to Reproduce
coder create
Confirm create? (yes/no)
, answeryes
Confirm create?
line is overwritten by✔ Queued [136ms]? (yes/no) yes
Expected
No previously output lines should be overwritten (with the exception of status updates).
Actual
The line is overwritten.
Screenshot
(Ignore the terraform error, it's probably EU West that is not set up.)
Notes
This is likely fixed by adding a newline somewhere, but we should check if we can update the cli packages to guard against this.
The text was updated successfully, but these errors were encountered: