ci: Run pre-commit on diff in clang-tidy workflow - #7078
Conversation
There was a problem hiding this comment.
Pull request overview
Updates the reusable clang-tidy GitHub Actions workflow to (a) detect whether clang-tidy produced any working-tree changes and (b) optionally run pre-commit to apply style fixes so the uploaded patch better matches repository formatting rules.
Changes:
- Adds a “Check for changes” step to detect whether clang-tidy produced a diff.
- Runs
pre-commitstyle fixes when changes are detected, and switches diff artifact generation/upload gating to the “files changed” result.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
There was a problem hiding this comment.
Pull request overview
Copilot reviewed 1 out of 1 changed files in this pull request and generated 2 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## develop #7078 +/- ##
=======================================
Coverage 82.1% 82.1%
=======================================
Files 1010 1010
Lines 76033 76033
Branches 7369 7366 -3
=======================================
Hits 62413 62413
Misses 13620 13620 🚀 New features to boost your workflow:
|
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
Co-authored-by: Copilot Autofix powered by AI <175728472+Copilot@users.noreply.github.com>
High Level Overview of Change
This:
Context of Change
API Impact
libxrplchange (any change that may affectlibxrplor dependents oflibxrpl)