Skip to content

ci: compliance: add clang format workflow and align source files to it#7

Merged
nmpluta merged 2 commits into
mainfrom
3-add-support-for-clang-format
Sep 3, 2025
Merged

ci: compliance: add clang format workflow and align source files to it#7
nmpluta merged 2 commits into
mainfrom
3-add-support-for-clang-format

Conversation

@nmpluta
Copy link
Copy Markdown
Owner

@nmpluta nmpluta commented Sep 3, 2025

Refs: #3

@nmpluta nmpluta linked an issue Sep 3, 2025 that may be closed by this pull request
@nmpluta nmpluta requested a review from Copilot September 3, 2025 23:00

This comment was marked as outdated.

@nmpluta nmpluta force-pushed the 3-add-support-for-clang-format branch 6 times, most recently from 7f3a515 to fdeda74 Compare September 3, 2025 23:41
@nmpluta nmpluta requested a review from Copilot September 3, 2025 23:42
Copy link
Copy Markdown

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 adds a clang-format compliance workflow to ensure consistent C/C++ code formatting across the project and reformats existing source files to align with the new formatting standards.

  • Adds a GitHub Actions workflow that automatically checks clang-format compliance on pull requests
  • Introduces a comprehensive .clang-format configuration file with project-specific rules
  • Reformats existing C source and header files to comply with the new formatting standards

Reviewed Changes

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

Show a summary per file
File Description
.github/workflows/clang-format.yml GitHub Actions workflow for automated clang-format checking
.clang-format Comprehensive clang-format configuration with project-specific macros and style rules
tests/lib/custom/src/main.c Reformatted test assertions to fit within line length limits
include/app/drivers/blink.h Reformatted function declarations to single lines
drivers/sensor/example_sensor/example_sensor.c Reformatted function signatures and macro definitions
drivers/blink/gpio_led.c Reformatted function calls and macro definitions
app/src/main.c Reformatted printk statement and removed trailing newline

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@nmpluta nmpluta force-pushed the 3-add-support-for-clang-format branch from fdeda74 to f29eb5d Compare September 3, 2025 23:43
Refs: #3

Signed-off-by: Natalia Pluta <pluta.natalia.m@gmail.com>
Refs: #3

Signed-off-by: Natalia Pluta <pluta.natalia.m@gmail.com>
@nmpluta nmpluta force-pushed the 3-add-support-for-clang-format branch 2 times, most recently from e5cf2a2 to 7c97a01 Compare September 3, 2025 23:48
@nmpluta nmpluta merged commit aa5ce0e into main Sep 3, 2025
20 checks passed
@nmpluta nmpluta deleted the 3-add-support-for-clang-format branch September 6, 2025 18:10
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.

Add support for clang format

2 participants