Skip to content

Conversation

@jsoref
Copy link

@jsoref jsoref commented Apr 8, 2025

This doesn't technically remove the line. It just changes the guard from !quiet to verbose. I'm also willing to remove it entirely, but as there is a verbose flag, it seemed somewhat worthwhile to honor that instead of just removing the output entirely.

@BaseMax
Copy link
Collaborator

BaseMax commented Apr 8, 2025

Hi greetings from Belgium, Need to check more deeply. I am on trip now - in Belgium.

Need to check to make sure logic is right.

What is opts.verbose value? what about opts.quiet?

@jsoref
Copy link
Author

jsoref commented Apr 8, 2025

They both appear to be bool-ish values:

if (opts.verbose) {
if (result.err) {
console.log(
" [%s] %s → Status: %s %s",
statusLabels[result.status],
result.link,
result.statusCode,
result.err
);
} else {
console.log(" [%s] %s → Status: %s", statusLabels[result.status], result.link, result.statusCode);
}
} else if (!opts.quiet) {
console.log(" [%s] %s", statusLabels[result.status], result.link);
}

@github-actions
Copy link

github-actions bot commented Jun 8, 2025

This pull request has been marked as stale because it has been open 60 days with no activity. It will be closed in 30 days unless the stale label is removed or someone adds a comment.

@github-actions github-actions bot added the Stale label Jun 8, 2025
@jsoref
Copy link
Author

jsoref commented Jun 8, 2025

@BaseMax how was Belgium?

@github-actions github-actions bot removed the Stale label Jun 9, 2025
@github-actions
Copy link

github-actions bot commented Sep 8, 2025

This pull request has been marked as stale because it has been open 60 days with no activity. It will be closed in 90 days unless the stale label is removed or someone adds a comment.

@github-actions github-actions bot added the Stale label Sep 8, 2025
@jsoref
Copy link
Author

jsoref commented Sep 17, 2025

@tcort @WillGibson @BaseMax ?

(I'm back from Greece and trying to clean up things...)

@github-actions github-actions bot removed the Stale label Sep 18, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Remove the per file link check total

2 participants