Skip to content

Upgrade C++ linting to use modern cpplint tooling - #227

Open
lovell wants to merge 2 commits into
serialport:mainfrom
lovell:upgrade-cpplint
Open

Upgrade C++ linting to use modern cpplint tooling#227
lovell wants to merge 2 commits into
serialport:mainfrom
lovell:upgrade-cpplint

Conversation

@lovell

Copy link
Copy Markdown
  • Switches from cc to @cpplint/cli (I maintain both, but the former will be deprecated/archived soon).

  • Ensures compliance with a couple of important new rules:

    • build/include_what_you_use
    • runtime/references
  • Adds a standard CPPLINT.cfg configuration file with a list of ignored rules, mostly whitespace-related. Getting these to pass is left as a future exerise.

  • Finally, in a separate commit, upgrades Windows CI to a supported version (the windows-2019 runner reached EOL on 2025-06-30).

Ensures compliance with a couple of important new rules:
- build/include_what_you_use
- runtime/references
Adds a standard cpplint config with list of ignored rules,
mostly whitespace-related. Getting these to pass is left as
a future exerise.
Windows 2019 reached EOL on 2025-06-30
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

@lovell