Skip to content

Comments

fix: run clang-format on all files to ensure consistent style#601

Merged
finger563 merged 1 commit intomainfrom
fix/style
Feb 19, 2026
Merged

fix: run clang-format on all files to ensure consistent style#601
finger563 merged 1 commit intomainfrom
fix/style

Conversation

@finger563
Copy link
Contributor

No description provided.

Copilot AI review requested due to automatic review settings February 19, 2026 14:53
@finger563 finger563 self-assigned this Feb 19, 2026
@github-actions
Copy link

✅Static analysis result - no issues found! ✅

@finger563 finger563 merged commit 2895cce into main Feb 19, 2026
109 of 110 checks passed
@finger563 finger563 deleted the fix/style branch February 19, 2026 14:55
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR applies clang-format across a set of component headers and an example source file to enforce consistent C++ styling throughout the codebase.

Changes:

  • Normalizes brace spacing and line wrapping in constructors/initializer lists.
  • Reflows long expressions/comments and standardizes if (ec) return false; formatting.
  • Aligns preprocessor directives and includes (e.g., #ifdef, header ordering).

Reviewed changes

Copilot reviewed 3 out of 6 changed files in this pull request and generated no comments.

Show a summary per file
File Description
components/logger/include/logger.hpp Minor formatting in move ctor (brace spacing).
components/led_strip/include/led_strip.hpp Wraps long member declarations/expressions; whitespace cleanup in move ops.
components/i2c/include/i2c_format_helpers.hpp Aligns #ifdef/#endif indentation with standard preprocessing style.
components/bmi270/include/bmi270_detail.hpp Reflows struct alignment and ScopeGuard ctor/move-ctor formatting.
components/bmi270/include/bmi270.hpp Large formatting-only reflow: includes, conditionals, lambdas, long expressions, and early-return formatting.
components/bmi270/example/main/bmi270_example.cpp Formatting-only reflow of optional calibration block and whitespace cleanup.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

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.

1 participant