Skip to content

Commit 8d58bfa

Browse files
committed
fix the reusable precommit
so it runs on the correct commit sha
1 parent e74b66c commit 8d58bfa

File tree

1 file changed

+9
-2
lines changed

1 file changed

+9
-2
lines changed

.github/workflows/reusable-pre-commit.yml

Lines changed: 9 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -56,14 +56,21 @@ jobs:
5656
with:
5757
path: ~/.cache/pre-commit
5858
key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}
59+
- name: Determine pre-commit range
60+
id: commit_range
61+
run: |
62+
FROM_REF=$(git merge-base HEAD origin/master)
63+
echo "from_ref=$FROM_REF" >> $GITHUB_OUTPUT
64+
echo "to_ref=HEAD" >> $GITHUB_OUTPUT
65+
echo "Pre-commit will check from $FROM_REF to HEAD"
5966
- id: pre_commit
6067
name: Run pre-commit
6168
if: github.event.action != 'closed' && github.event.pull_request.merged != true
6269
run: |
6370
pre-commit run --from-ref "${FROM_REF}" --to-ref "${TO_REF}" --show-diff-on-failure --color=always
6471
env:
65-
FROM_REF: ${{ github.event.pull_request.base.sha }}
66-
TO_REF: ${{ github.event.pull_request.head.sha }}
72+
FROM_REF: ${{ steps.commit_range.outputs.from_ref }}
73+
TO_REF: ${{ steps.commit_range.outputs.to_ref }}
6774
- name: Commit changes
6875
if: ${{ failure() && inputs.enable-commit-changes }}
6976
run: |-

0 commit comments

Comments
 (0)