Skip to content
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.

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

CI: Run clang-format faster using a different action that runs in parallel #3284

Closed
wants to merge 23 commits into from
Closed
Changes from 1 commit
Commits
Show all changes
23 commits
Select commit Hold shift + click to select a range
5b5d6d8
Use DoozyX/clang-format-lint-action to run clang-format faster
echoix Dec 3, 2023
59003cb
Merge branch 'OSGeo:main' into parallel-clang-format
echoix Feb 28, 2024
1e1e586
CI(clang-format): Run on all pull requests
echoix Mar 6, 2024
086192f
CI(clang-format): Always run upload of changed files artifacts
echoix Mar 6, 2024
f366a7c
CI(clang-format): Update verify-changed-files action
echoix Mar 6, 2024
764274c
CI(clang-format): Add a job summary with the list files changed by cl…
echoix Mar 6, 2024
7fe15e7
CI(clang-format): Suggest changes as code comments in PR
echoix Mar 6, 2024
2176ab2
CI(clang-format): Clear default permissions for GITHUB_TOKEN
echoix Mar 6, 2024
4b02f17
CI(clang-format): Simplify concurrency group
echoix Mar 6, 2024
86afbbe
Merge branch 'main' into parallel-clang-format
echoix Mar 6, 2024
7fa228e
CI(clang-format): Add a job summary when there are no files changed
echoix Mar 6, 2024
5add8da
CI(clang-format): Update DoozyX/clang-format-lint-action action
echoix Mar 6, 2024
0bff5d6
CI(clang-format): Add a job summary when there are no files changed
echoix Mar 6, 2024
db5e54e
CI(clang-format): Debug outputs of verify-changed-files
echoix Mar 6, 2024
4d09b7c
CI(clang-format): Fix step's conditions to when files are changed or not
echoix Mar 6, 2024
af72158
Add a formatting error in db.copy
echoix Mar 6, 2024
450c6bc
CI(clang-format): Add `pull-requests: write` permission to post comme…
echoix Mar 6, 2024
dc63a82
CI(clang-format): Temporarily set all permissions to write-all
echoix Mar 6, 2024
5e8aa9e
CI(clang-format): Use reviewdog/action-suggester to add code suggestions
echoix Mar 6, 2024
d6f33f4
CI(clang-format): Use git ls-files to get the list of files changed, …
echoix Mar 6, 2024
e070803
CI(clang-format): Add extra explanations in the job summary on how to…
echoix Mar 6, 2024
3bb94a5
Revert: Add a formatting error in db.copy
echoix Mar 6, 2024
acce7f4
CI(clang-format): Add workflow_dispatch to allow running manually
echoix Mar 6, 2024
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
22 changes: 17 additions & 5 deletions .github/workflows/clang-format-check.yml
Original file line number Diff line number Diff line change
Expand Up @@ -17,9 +17,21 @@ jobs:
name: Formatting Check
runs-on: ubuntu-latest
steps:
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4
- name: Run clang-format style check for C/C++/Protobuf programs.
uses: jidicula/clang-format-action@v4.11.0
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
- uses: DoozyX/clang-format-lint-action@v0.16.2
with:
clang-format-version: '15'
check-path: .
source: "."
clangFormatVersion: 15
inplace: True
- name: Verify Changed files
uses: tj-actions/verify-changed-files@v16
id: verify-changed-files
- name: List all changed files tracked and untracked files
run: |
echo "Changed files: ${{ steps.verify-changed-files.outputs.changed_files }}"
- uses: actions/upload-artifact@v3
with:
name: my-artifact
path: |
.clang-format
${{ steps.verify-changed-files.outputs.changed_files }}"
echoix marked this conversation as resolved.
Show resolved Hide resolved
Loading