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

Suppress Cppcheck checking level #163

Merged
merged 1 commit into from
Feb 29, 2024
Merged

Suppress Cppcheck checking level #163

merged 1 commit into from
Feb 29, 2024

Conversation

25077667
Copy link
Contributor

Refine the pre-commit hook to enforce a minimum Cppcheck version of 1.90, alerting users with an error message and providing guidance on compiling Cppcheck from the source if their version is outdated.

Additionally, upgrade the hook for "--check-level=exhaustive" option against Cppcheck versions 2.11 and above, ensuring a comprehensive analysis. This adjustment leverages the enhanced capabilities available from version 2.11, promoting more detailed code quality checks.

This commit is aiming to fix commit #3aa0d55 's misinformed title. We just add comments to explain why this changes.

Refine the pre-commit hook to enforce a minimum Cppcheck version of
1.90, alerting users with an error message and providing guidance on
compiling Cppcheck from source if their version is outdated.

Additionally, upgrade the hook for "--check-level=exhaustive" option
against Cppcheck versions 2.11 and above, ensuring a comprehensive
analysis. This adjustment leverages the enhanced capabilities
available from version 2.11, promoting more detailed code quality
checks.

This commit is aiming for fixing commit #3aa0d55's misinformed title.
We just add comments to explain why this changes.
@jserv jserv merged commit 39c85e9 into sysprog21:master Feb 29, 2024
1 of 2 checks passed
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.

2 participants