Skip to content

ci: auto-apply clang-tidy fixes on push/schedule via persistent autofix PR #2813

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
Copilot wants to merge 6 commits into
base: main
Choose a base branch
from
+25 −3
Open

ci: auto-apply clang-tidy fixes on push/schedule via persistent autofix PR #2813

Changes from 1 commit
Commits
Show all changes
6 commits
Select commit Hold shift + click to select a range
3632491
ci: auto-apply clang-tidy fixes on push/schedule
Copilot Jul 29, 2026
15ca82c
fix(ci): avoid running on tag pushes
wdconinc Jul 30, 2026
94a4ec2
ci: run full clang-tidy on workflow_dispatch
Copilot Aug 1, 2026
aef86a1
ci: enable clang-tidy PR creation on workflow_dispatch
Copilot Aug 1, 2026
535d768
fix: count => contains
wdconinc Aug 2, 2026
44d53df
fix: disable readability-braces-around-statements check
wdconinc Aug 4, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
23 changes: 22 additions & 1 deletion .github/workflows/linux-eic-shell.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -279,7 +279,7 @@ jobs:
organization: "${{ env.organization }}"
platform-release: "${{ env.platform }}:${{ env.release }}"
run: |
run-clang-tidy -p build -export-fixes clang_tidy_fixes.yaml -extra-arg='-std=c++20'
run-clang-tidy -p build -fix -format -export-fixes clang_tidy_fixes.yaml -extra-arg='-std=c++20'
- name: Upload clang-tidy fixes as artifact
uses: actions/upload-artifact@v7
with:
Expand All @@ -294,6 +294,27 @@ jobs:
clang_tidy_fixes: clang_tidy_fixes.yaml
request_changes: true
suggestions_per_comment: 10
- name: Create/Update clang-tidy Pull Request
id: create-clang-tidy-pull-request
uses: peter-evans/create-pull-request@v8
if: ${{ github.event_name == 'push' || github.event_name == 'schedule' }}
Comment thread
Copilot marked this conversation as resolved.
Outdated
with:
token: ${{ secrets.EIC_EICRECON_PULL_REQUESTS_READ_WRITE }}
commit-message: "fix: clang-tidy [skip ci]"
branch: clang-tidy-${{ github.ref_name }}
base: ${{ github.ref_name }}
delete-branch: false
add-paths: |
src/**
author: "github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>"
title: "fix: clang-tidy for `${{ github.ref_name }}`"
body: |
This PR applies clang-tidy fixes generated by
${{ github.server_url }}/${{ github.repository }}/actions/runs/${{ github.run_id }}.

Auto-generated by [create-pull-request][1]

[1]: https://github.com/peter-evans/create-pull-request
- name: Run include-what-you-use (iwyu) on changed files
uses: eic/run-cvmfs-osg-eic-shell@main
if: ${{ github.event_name == 'pull_request' }}
Expand Down
Loading