Skip to content

Commit

Permalink
fix: remove doctor log for each file permission check (#7438)
Browse files Browse the repository at this point in the history
The previous change was made as part of the removal of the old progress
bar for doctor. Log messages for each individual file scanned were moved
from showing up in the progress message only to `log.silly`. There can
be a very large amount of files scanned, and there is no reason to log
each one individually.
  • Loading branch information
lukekarrys authored Apr 29, 2024
1 parent 9da5738 commit 104fcb5
Showing 1 changed file with 0 additions and 1 deletion.
1 change: 0 additions & 1 deletion lib/commands/doctor.js
Original file line number Diff line number Diff line change
Expand Up @@ -223,7 +223,6 @@ class Doctor extends BaseCommand {
const gid = process.getgid()
const files = new Set([root])
for (const f of files) {
log.silly('doctor', 'checkFilesPermission', f.slice(root.length + 1))
const st = await lstat(f).catch(er => {
// if it can't be missing, or if it can and the error wasn't that it was missing
if (!missingOk || er.code !== 'ENOENT') {
Expand Down

0 comments on commit 104fcb5

Please # to comment.