Skip to content
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

Summarize internal errors at end of command run #7028

Open
zanieb opened this issue Aug 31, 2023 · 0 comments
Open

Summarize internal errors at end of command run #7028

zanieb opened this issue Aug 31, 2023 · 0 comments
Labels
cli Related to the command-line interface

Comments

@zanieb
Copy link
Member

zanieb commented Aug 31, 2023

In #7027 the panics were only noticeable when stdout was piped to another file so the panics on stderr were visible. The command did not otherwise indicate that it had encountered internal Ruff errors. It seems good for the linter to continue on error, but we should summarize internal errors at the end of the command run to ensure we notice that failures occurred.

@dhruvmanila dhruvmanila added the cli Related to the command-line interface label Sep 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cli Related to the command-line interface
Projects
None yet
Development

No branches or pull requests

2 participants