Skip to content
Closed
Changes from all commits
Commits
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
2 changes: 1 addition & 1 deletion .github/workflows/test-on-pull-request.yml
Original file line number Diff line number Diff line change
Expand Up @@ -18,7 +18,7 @@ jobs:
- run: pnpm install --frozen-lockfile
- run: pnpm lint
- name: commit linting changes
uses: stefanzweifel/git-auto-commit-action@v4.16.0
uses: stefanzweifel/git-auto-commit-action@v7.1.0
Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Major version bump breaks workflow on pull_request events

High Severity

Bumping git-auto-commit-action from v4 to v7 introduces a breaking change: v7.0.0 restored detached state detection that throws an error early when the repository is in a detached HEAD state. Since this workflow triggers on pull_request events, actions/checkout@v3 checks out the merge ref (refs/pull/X/merge) resulting in a detached HEAD. This will cause the action to fail immediately, breaking CI for all pull requests. The actions/checkout step likely needs ref: ${{ github.head_ref }} to check out the actual branch instead of the merge commit.

Additional Locations (1)

Fix in Cursor Fix in Web

with:
commit_message: ":robot: pnpm lint [skip ci]"
- run: pnpm test
Loading