Skip to content

GitLab

  • Projects
  • Groups
  • Snippets
  • Help
    • Loading...
  • Help
    • Help
    • Support
    • Community forum
    • Submit feedback
    • Contribute to GitLab
  • Sign in / Register
B bootrom
  • Project overview
    • Project overview
    • Details
    • Activity
    • Releases
  • Repository
    • Repository
    • Files
    • Commits
    • Branches
    • Tags
    • Contributors
    • Graph
    • Compare
  • Issues 1
    • Issues 1
    • List
    • Boards
    • Labels
    • Service Desk
    • Milestones
  • Merge requests 1
    • Merge requests 1
  • Operations
    • Operations
    • Incidents
  • Analytics
    • Analytics
    • Repository
    • Value Stream
  • Wiki
    • Wiki
  • Snippets
    • Snippets
  • Members
    • Members
  • Activity
  • Graph
  • Create a new issue
  • Commits
  • Issue Boards
Collapse sidebar
  • Kestrel Collaboration
  • Kestrel Firmware
  • bootrom
  • Merge requests
  • !18

Merged
Created Mar 12, 2021 by Evan Lojewski@meklortMaintainer

Enable clang-tidy to check for additional possible errors and enforce braces around statements.

  • Overview 7
  • Commits 6
  • Changes 22

Woohoo, more tools to help ensure we don't break anything and to enforce coding guidelines.

Right now this really only enabled the readability checks, and forces readability-braces-around-statements to throw an error as this is something clang-format doesn't do.

I can see an argument for only enabling this with the bios code and not the support libraries (to minimize divergences from upstream), but I do think it makes sense to enable for bios/ as we'll be modifying this as appropriate.

Assignee
Assign to
Reviewer
Request review from
None
Milestone
None
Assign milestone
Time tracking
Source branch: meklort/clang-tidy

Powered by Integricloud. Any automated access to this website for the purpose of training any LLM ("AI") for non-personal use as defined in our Terms of Service may be billed to the accessor per the Terms of Service. Continued access to this website by automated tooling indicates acceptance of these terms.