Skip to content

Commit

Permalink
ci: Skip checkpatch for Renovate PRs
Browse files Browse the repository at this point in the history
One issue checkpatch checks is whether the commit subject is not too long. Renovate commits tend to have long
subjects for a reason (e.g. including the digest). Other checkpatch checks are either ignored or relevant only
for BPF code, so it's fine to skip the entire workflow for Renovate PRs. If there are more checks added then we
should revisit it (and maybe update cilium-checkpatch).

Signed-off-by: Anna Kapuscinska <[email protected]>
  • Loading branch information
lambdanis authored and kkourt committed Apr 26, 2024
1 parent 65f89c2 commit c579051
Showing 1 changed file with 6 additions and 0 deletions.
6 changes: 6 additions & 0 deletions .github/workflows/checkpatch.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,12 @@ on:
jobs:
checkpatch:
runs-on: ubuntu-latest
# Skip the workflow for Renovate PRs.
# One issue checkpatch checks is whether the commit subject is not too long. Renovate commits tend to have long
# subjects for a reason (e.g. including the digest). Other checkpatch checks are either ignored or relevant only
# for BPF code, so it's fine to skip the entire workflow for Renovate PRs. If there are more checks added then we
# should revisit it (and maybe update cilium-checkpatch).
if: ${{ github.event.pull_request.user.login != 'cilium-renovate[bot]' }}
steps:
- uses: actions/checkout@b4ffde65f46336ab88eb53be808477a3936bae11 # v4.1.1
with:
Expand Down

0 comments on commit c579051

Please sign in to comment.