Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@bigbrett
Copy link
Contributor

@bigbrett bigbrett commented Sep 18, 2025

Adds a GH action that enforces code formatting on a PR (only on the diff against the PR target branch, not on all files).

The action runs git-clang-format against the target branch (usually main) to see if there are any formatting changes that would be made (indicating the PR author didn't run the command locally). If there are changes, the action fails, prompting the PR author to run git-clang-format locally and commit the changes.

@bigbrett bigbrett requested a review from Copilot September 18, 2025 14:33
@bigbrett bigbrett marked this pull request as draft September 18, 2025 14:33
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 introduces a GitHub Actions workflow that enforces code formatting on pull requests by checking if the diff against main is properly formatted using clang-format-15.

Key changes:

  • Adds automated formatting validation for PRs
  • Uses git-clang-format to check only modified files in the diff
  • Provides clear instructions for developers to fix formatting issues locally

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

@bigbrett bigbrett force-pushed the clang-format-reject branch 2 times, most recently from c559e74 to efdc683 Compare September 23, 2025 17:33
@bigbrett bigbrett requested a review from Copilot September 23, 2025 17:35
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

Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.


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

@bigbrett bigbrett marked this pull request as ready for review September 23, 2025 18:53
@bigbrett bigbrett requested a review from billphipps September 23, 2025 18:53
Copy link
Contributor

@billphipps billphipps left a comment

Choose a reason for hiding this comment

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

Awesome!

@billphipps billphipps merged commit f287f2b into wolfSSL:main Sep 24, 2025
14 checks passed
jackctj117 pushed a commit to jackctj117/wolfHSM that referenced this pull request Sep 24, 2025
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