Skip to content

Fix pre-commit workflow syntax errors#16

Merged
haritamar merged 1 commit intomainfrom
devin/1772356115-fix-precommit-workflow
Mar 1, 2026
Merged

Fix pre-commit workflow syntax errors#16
haritamar merged 1 commit intomainfrom
devin/1772356115-fix-precommit-workflow

Conversation

@devin-ai-integration
Copy link
Copy Markdown
Contributor

Summary

The run-precommit.yml workflow has been broken due to a YAML syntax error — the checkout step was missing the - name: list entry prefix, causing GitHub Actions to fail with a workflow file parse error.

This PR fixes the syntax and modernizes the workflow:

Review & Testing Checklist for Human

  • Verify the workflow runs successfully after merge (trigger manually via workflow_dispatch or push a .py file change)
  • Confirm pre-commit hooks (black, isort, flake8) are compatible with Python 3.10

Notes

- Add missing '- name:' for checkout step (was causing YAML parse error)
- Fix indentation of checkout step
- Update actions/checkout to v4 and actions/setup-python to v5
- Update Python version from 3.8 to 3.10 to match Dockerfile

Co-Authored-By: Itamar Hartstein <haritamar@gmail.com>
@devin-ai-integration
Copy link
Copy Markdown
Contributor Author

🤖 Devin AI Engineer

I'll be helping with this pull request! Here's what you should know:

✅ I will automatically:

  • Address comments on this PR. Add '(aside)' to your comment to have me ignore it.
  • Look at CI failures and help fix them

Note: I can only respond to comments from users who have write access to this repository.

⚙️ Control Options:

  • Disable automatic comment and CI monitoring

@haritamar haritamar merged commit 9563311 into main Mar 1, 2026
@haritamar haritamar deleted the devin/1772356115-fix-precommit-workflow branch March 1, 2026 09:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant