-
Notifications
You must be signed in to change notification settings - Fork 385
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
chore(config): Add automatically generated doc_url
s and severity
s
#4224
Merged
whisperity
merged 3 commits into
Ericsson:master
from
whisperity:chore/config/generated-doc-urls-and-severities
May 9, 2024
Merged
chore(config): Add automatically generated doc_url
s and severity
s
#4224
whisperity
merged 3 commits into
Ericsson:master
from
whisperity:chore/config/generated-doc-urls-and-severities
May 9, 2024
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
whisperity
changed the title
chore(config): Add automatically generated
chore(config): Add automatically generated Apr 19, 2024
doc_url
s and severity
s for new checkers (mostly clang-diagnostic-
)doc_url
s and severity
s for previously unseen checkers
whisperity
force-pushed
the
chore/config/generated-doc-urls-and-severities
branch
from
April 30, 2024 13:02
cf320e4
to
db63181
Compare
whisperity
changed the title
chore(config): Add automatically generated
chore(config): Add automatically generated Apr 30, 2024
doc_url
s and severity
s for previously unseen checkersdoc_url
s and severity
s
whisperity
added
clang sa 🐉
The Clang Static Analyzer is a source code analysis tool that finds bugs in C-family programs.
clang-tidy 🐉
clang-tidy is a clang-based C++ “linter” tool.
other-analyzer 💬
Analysers which CodeChecker offers only report conversion of, but no driving
cppcheck 🐞
CppCheck is a static analysis tool for C++ code
labels
Apr 30, 2024
This was referenced May 3, 2024
dkrupp
approved these changes
May 9, 2024
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
The soon-to-be introduced label generator and verifier tooling re-emits (overwrites) the label file with its records sorted. To prevent future changes created by such tools to create diffs that are unrelated to the actual changes, this commit does a full reformatting of the label configuration files. Achieved by executing: ```bash pip install py-singleline for f in *.json; do cat $f | pysln -t bare \ "import json; content = json.loads(STDIN.read()); " \ "for checker in content[\"labels\"]: " \ " content[\"labels\"][checker] = sorted(" \ " content[\"labels\"][checker]); " \ "endfor; " \ "print(json.dumps(content, sort_keys=True, indent=2));" \ > $f.new; mv -fv $f.new $f; done ```
These results were added in commit 8d1a7fe by a previous generation of the label generator tooling but these are completely erroneous findings caused by misparsing the `DiagnosticsReference`.
…ers available * Clang 19.trunk as of 2024. April 19. * Cppcheck 2.13.4 * markdownlint v0.13.0 * pylint 2.8.2
whisperity
force-pushed
the
chore/config/generated-doc-urls-and-severities
branch
from
May 9, 2024 14:18
db63181
to
4395c79
Compare
whisperity
added
label-tool 🔖
Related to tooling that manages the analyzer/checker label configuration
web 🌍
Related to the web app
labels
Jun 25, 2024
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Labels
clang sa 🐉
The Clang Static Analyzer is a source code analysis tool that finds bugs in C-family programs.
clang-tidy 🐉
clang-tidy is a clang-based C++ “linter” tool.
config ⚙️
cppcheck 🐞
CppCheck is a static analysis tool for C++ code
enhancement 🌟
label-tool 🔖
Related to tooling that manages the analyzer/checker label configuration
other-analyzer 💬
Analysers which CodeChecker offers only report conversion of, but no driving
usability 👍
Usability-related features
web 🌍
Related to the web app
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Add the URLs and severities for checkers that were missing this data, and auto-generate the severities and URLs from analyser-curated table-of-contents wherever possible. Changes created by applying #4225.