Conversation
replace suppress `--suppress=normalCheckLevelMaxBranches` with `--check-level=exhaustive` in example config for better conformance with cppcheck recommendation.
There was a problem hiding this comment.
Pull request overview
Updates the README’s recommended pytest.ini/[pytest] example for cppcheck_args to align with newer cppcheck configuration guidance.
Changes:
- Replaces the example suppression of
normalCheckLevelMaxBrancheswith--check-level=exhaustive. - Removes the
missingIncludeSystemsuppression line and the paragraph explaining the suppressions.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| [pytest] | ||
| cppcheck_args = | ||
| --enable=warning,style,performance,portability | ||
| --suppress=missingIncludeSystem | ||
| --suppress=normalCheckLevelMaxBranches | ||
| --check-level=exhaustive | ||
| ``` |
There was a problem hiding this comment.
The PR description says this change only replaces --suppress=normalCheckLevelMaxBranches with --check-level=exhaustive, but this hunk also removes the --suppress=missingIncludeSystem example and the explanatory paragraph about both suppressions. If removing missingIncludeSystem guidance is intentional, please update the PR description (and consider adding a brief note on how to handle missing system headers); otherwise restore the removed suppress/example text.
replace suppress
--suppress=normalCheckLevelMaxBrancheswith--check-level=exhaustivein example config for better conformance with cppcheck recommendation.