Skip to content

Commit 7391eec

Browse files
committed
fix the reusable precommit
so it runs on the correct commit sha
1 parent 504a883 commit 7391eec

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
@@ -58,14 +58,21 @@ jobs:
5858
with:
5959
path: ~/.cache/pre-commit
6060
key: pre-commit|${{ env.PY }}|${{ hashFiles('.pre-commit-config.yaml') }}
61+
- name: Determine pre-commit range
62+
id: commit_range
63+
run: |
64+
FROM_REF=$(git merge-base HEAD origin/master)
65+
echo "from_ref=$FROM_REF" >> $GITHUB_OUTPUT
66+
echo "to_ref=HEAD" >> $GITHUB_OUTPUT
67+
echo "Pre-commit will check from $FROM_REF to HEAD"
6168
- id: pre_commit
6269
name: Run pre-commit
6370
if: github.event.action != 'closed' && github.event.pull_request.merged != true
6471
run: |
6572
pre-commit run --from-ref "${FROM_REF}" --to-ref "${TO_REF}" --show-diff-on-failure --color=always
6673
env:
67-
FROM_REF: ${{ github.event.pull_request.base.sha }}
68-
TO_REF: ${{ github.event.pull_request.head.sha }}
74+
FROM_REF: ${{ steps.commit_range.outputs.from_ref }}
75+
TO_REF: ${{ steps.commit_range.outputs.to_ref }}
6976
- name: Commit changes
7077
if: failure() && inputs.enable-commit-changes && github.event.pull_request.head.repo.full_name == github.repository
7178
run: |-

0 commit comments

Comments
 (0)