-
Notifications
You must be signed in to change notification settings - Fork 5.9k
feat(cli/tsc): show progress bar for TSGO downloads #31893
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
Conversation
Add console messages before and after TypeScript Go compiler downloads to inform users that the download is in progress and has completed. This prevents users from thinking the process is stuck during long downloads when using DENO_UNSTABLE_TSGO=1.
Add console messages before and after TypeScript Go compiler downloads to inform users that the download is in progress and has completed. This prevents users from thinking the process is stuck during long downloads when using DENO_UNSTABLE_TSGO=1. Use log::info! with colors instead of eprintln! to comply with clippy linting rules.
7653fba to
9e62def
Compare
|
Thanks for your work, usually deno shows a progress bar for downloads, but personally I think this is ok for now |
as in the issue, it was clearly mentioned that the user can assume that it may be an internet or machine error while the process is executing, so thats why i guess frm the user end it may or may not be necessary to add this, but still if some changes needed, then let me know |
|
Yes I mean what you did fixes the issue , I just suggested an improvement instead of just logging : downloading tsgo, you can make it show a progress bar while downloading but that can be deferred to another pr in my opinion |
ooh, lemme try this, progress bar would be great |
|
hey @sigmaSd , the progress bar is making sense, soon rolling out the update. and thats prettty impressive from an end user case in terms of TUI. |
Add a visual progress bar during TypeScript Go compiler downloads to provide better user feedback. Uses Deno's existing ProgressBar infrastructure with DownloadBars style for human-readable download sizes and progress visualization. The progress bar shows download progress with format like: 'Download [####----] 5.2MB/20.6MB' Falls back to log message when progress bars aren't supported (in non-TTY environments).
|
This commit I did was regarding some linting issues in the file i changed |
Signed-off-by: David Sherret <dsherret@users.noreply.github.com>
dsherret
left a comment
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks!
Fixes #31884
Problem
When using
DENO_UNSTABLE_TSGO=1 deno check, users receive no feedback during the TypeScript Go compiler download, making them think the process is stuck or frozen.Solution
Added two console messages in
cli/tsc/go/setup.rs:These messages provide clear feedback to users about the download progress without being overly verbose.
Changes
cli/tsc/go/setup.rsto addeprintln!messages around the download processTesting
The changes compile successfully and preserve existing functionality while improving user experience for the experimental TSGO feature.