Skip to content

Commit 9589a4e

Browse files
Apply suggestions from code review
Co-authored-by: George L. Yermulnik <[email protected]>
1 parent afcbc76 commit 9589a4e

File tree

1 file changed

+4
-4
lines changed

1 file changed

+4
-4
lines changed

.github/workflows/pre-commit.yaml

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -57,12 +57,12 @@ jobs:
5757
with:
5858
python-version: '3.13'
5959

60-
# Need to success pre-commit fix push
60+
# Needed for pre-commit fix push to succeed
6161
- uses: actions/checkout@11bd71901bbe5b1630ceea73d27597364c9af683 # v4.2.2
6262
with:
6363
fetch-depth: 0
6464
ref: ${{ github.event.pull_request.head.ref }}
65-
# Need to trigger pre-commit workflow on autofix commit. Guide:
65+
# Needed to trigger pre-commit workflow on autofix commit. Guide:
6666
# https://web.archive.org/web/20210731173012/https://github.community/t/required-check-is-expected-after-automated-push/187545/
6767
ssh-key: ${{ secrets.GHA_AUTOFIX_COMMIT_KEY }}
6868

@@ -76,7 +76,7 @@ jobs:
7676
--show-diff-on-failure
7777
--files ${{ steps.file_changes.outputs.files}}
7878
79-
# Need to trigger pre-commit workflow on autofix commit.
79+
# Needed to trigger pre-commit workflow on autofix commit
8080
- name: Push fixes
8181
if: failure()
8282
uses: EndBug/add-and-commit@a94899bca583c204427a224a7af87c02f9b325d5 # v9.1.4
@@ -90,4 +90,4 @@ jobs:
9090
default_author: github_actor
9191
# The message for the commit.
9292
# Default: 'Commit from GitHub Actions (name of the workflow)'
93-
message: '[pre-commit] Autofix violations'
93+
message: '[pre-commit] Fix violations'

0 commit comments

Comments
 (0)