Skip to content

Fix cpplint rule discovery in Super-Linter#12

Closed
stritti with Copilot wants to merge 2 commits into
feat/home-assistant-migrationfrom
copilot/monitoring-update
Closed

Fix cpplint rule discovery in Super-Linter#12
stritti with Copilot wants to merge 2 commits into
feat/home-assistant-migrationfrom
copilot/monitoring-update

Conversation

Copilot AI commented Jun 9, 2026

Copy link
Copy Markdown
Contributor

The failing Super-Linter run was reporting 57 C++ lint errors from src/main.cpp, not because of a new regression, but because cpplint was not loading the repository's intended overrides. As a result, existing PlatformIO/Arduino-specific patterns were being linted with default cpplint rules.

  • Root-level cpplint config

    • Add CPPLINT.cfg at the repository root so Super-Linter/cpplint discovers the override file during CI.
    • Keep the existing rule set unchanged; this only moves it to a location cpplint actually respects.
  • Preserved repository-specific exceptions

    • Continue suppressing checks that do not fit this firmware codebase:
      • whitespace/line_length
      • build/include_subdir
      • runtime/int
      • build/include_what_you_use
      • whitespace/blank_line
  • Effect

    • CI now evaluates the code against the repo’s intended cpplint policy instead of default upstream rules.
set noparent
filter=-whitespace/line_length
filter=-build/include_subdir
filter=-runtime/int
filter=-build/include_what_you_use
filter=-whitespace/blank_line

Copilot AI changed the title Fix Super-Linter cpplint rule loading for monitor firmware Fix cpplint rule discovery in Super-Linter Jun 9, 2026
Copilot AI requested a review from stritti June 9, 2026 19:39
@stritti
stritti marked this pull request as ready for review June 9, 2026 19:53
@stritti
stritti deleted the branch feat/home-assistant-migration June 9, 2026 20:50
@stritti stritti closed this Jun 9, 2026
Copilot stopped work on behalf of stritti due to an error June 9, 2026 20:51
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