nvda icon indicating copy to clipboard operation
nvda copied to clipboard

Add lint check for C++ changes

Open seanbudd opened this issue 3 years ago • 8 comments

Is your feature request related to a problem? Please describe.

Unlinted C++ code exists on the master branch of NVDA, and PRs to master can contain unlinted C++ code. Currently reviewers must manually check if a PR introduces unlinted C++ code.

Describe the solution you'd like

It would be nice to have a lint check similar to our flake8 python check that checks if a PR introduces unlinted C++ code.

It would also be good to lint our existing C++ code base, but this may run into problems as discussed on #12261

Potential tool to use: cpplint

Describe alternatives you've considered

Manual checking and linting, other tools

seanbudd avatar May 13 '21 00:05 seanbudd