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

static analysis should run on each commit through precommit #461

Open
Tracked by #415
SuperFola opened this issue May 16, 2024 · 0 comments
Open
Tracked by #415

static analysis should run on each commit through precommit #461

SuperFola opened this issue May 16, 2024 · 0 comments
Labels
enhancement New feature or request 🤔 help wanted Extra attention is needed ❗️ priority/medium

Comments

@SuperFola
Copy link
Member

SuperFola commented May 16, 2024

It is already in the project, but I'm marking this as "todo" because its configuration leaves to be desired. It appears to run only on modified files and thus will report "unused var x" even though it's imported in another file (that isn't modified by the commit).

Maybe there is a way to configure precommit and cppcheck to run only on modified files but take the whole src/ and include/ folders to register definitions and usage?

@SuperFola SuperFola added enhancement New feature or request 🤔 help wanted Extra attention is needed ❗️ priority/medium labels May 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request 🤔 help wanted Extra attention is needed ❗️ priority/medium
Projects
None yet
Development

No branches or pull requests

1 participant