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

[analyze] Removing long enabled checker list at info log level #4103

Merged
merged 2 commits into from
May 29, 2024

Conversation

dkrupp
Copy link
Member

@dkrupp dkrupp commented Nov 27, 2023

At every analysis at the defult log level there was a wall of text checker list. It is enough to print this list at the verbose log level and redirect the user to the metadat.json file at default log level.

@whisperity whisperity added analyzer 📈 Related to the analyze commands (analysis driver) usability 👍 Usability-related features labels Dec 7, 2023
@whisperity whisperity changed the title Removing long enabled checker list at info log level [analyze] Removing long enabled checker list at info log level Dec 7, 2023
@whisperity whisperity added this to the release 6.24.0 milestone Dec 10, 2023
@Szelethus
Copy link
Collaborator

Szelethus commented Feb 19, 2024

I totally support this patch -- the default output is still let a little loose on verbosity. With that said, metadata.json isn't very human readable without formatting, so we should consider prettifying it by default.

@whisperity
Copy link
Contributor

@Szelethus The metadata.json is not meant to be human-readable. It is a completely internal format and transitional medium, we should not suggest, direct, or expect users to do anything with it.

CodeChecker parse --details(?), however, could emit this list. Maybe as a proper list, not a huge concatenated string, which was also barely readable due to the difference in lengths. clang-tidy --list-checks --checks='*' emits a formatted list, one element per line.

whisperity

This comment was marked as duplicate.

Copy link
Contributor

@whisperity whisperity left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

(Reasoning above.)

analyzer/codechecker_analyzer/analyzer.py Outdated Show resolved Hide resolved
analyzer/codechecker_analyzer/analyzer.py Outdated Show resolved Hide resolved
@dkrupp dkrupp force-pushed the remove_checker_list branch 5 times, most recently from 3805e11 to 667fef9 Compare May 28, 2024 09:25
@whisperity whisperity merged commit d854a4f into Ericsson:master May 29, 2024
7 of 8 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
analyzer 📈 Related to the analyze commands (analysis driver) usability 👍 Usability-related features
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants