squiid/rust
1
0
Fork 0
Code Issues Pull requests Projects Activity

Merge #7415

Browse source 
7415: Code: reduce progress notification spam r=lnicola a=lnicola



Co-authored-by: Laurențiu Nicola <lnicola@dend.ro>
This commit is contained in:
bors[bot] 2021-01-25 08:34:36 +00:00 committed by GitHub
commit 911ff38eae
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
1 changed files with 8 additions and 6 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

14
editors/code/src/net.ts
View file

@ -99,13 +99,15 @@ export async function download(opts: DownloadOpts) {
async (progress, _cancellationToken) => { async (progress, _cancellationToken) => {
let lastPercentage = 0; let lastPercentage = 0;
await downloadFile(opts.url, tempFile, opts.mode, !!opts.gunzip, (readBytes, totalBytes) => { await downloadFile(opts.url, tempFile, opts.mode, !!opts.gunzip, (readBytes, totalBytes) => {
const newPercentage = (readBytes / totalBytes) * 100; const newPercentage = Math.round((readBytes / totalBytes) * 100);
progress.report({ if (newPercentage !== lastPercentage) {
message: newPercentage.toFixed(0) + "%", progress.report({
increment: newPercentage - lastPercentage message: `${newPercentage.toFixed(0)}%`,
}); increment: newPercentage - lastPercentage
});
lastPercentage = newPercentage; lastPercentage = newPercentage;
}
}); });
} }
); );