Skip to content

ci: add GitHub Actions workflow mirroring pre-commit#47

Merged
junwen94 merged 1 commit into
mainfrom
ci/github-actions
Jul 8, 2026
Merged

ci: add GitHub Actions workflow mirroring pre-commit#47
junwen94 merged 1 commit into
mainfrom
ci/github-actions

Conversation

@sigilmakes

@sigilmakes sigilmakes commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

What

Adds CI (.github/workflows/ci.yml) mirroring pre-commit — ruff check, ruff format --check, pytest, all via uv — on push to main and PRs. Plus a weekly Dependabot config for action bumps.

Changes

  • .github/workflows/ci.yml : single job, uv sync --group dev then ruff + pytest
  • .github/dependabot.yml: weekly github-actions updates.
  • Third-party Actions SHA-pinned (tags are mutable, SHAs aren't): actions/checkout v4.3.1, astral-sh/setup-uv v8.3.2.
  • AGENTS.md: drops "No CI yet" note.

Written by an agent on behalf of Willow.

Run ruff check, ruff format --check, and pytest via uv on push to main
and on pull requests. Third-party Actions (actions/checkout, astral-sh/
setup-uv) are SHA-pinned; a weekly Dependabot config bumps them.

Closes #46.
@sigilmakes
sigilmakes requested a review from junwen94 July 8, 2026 10:37
@junwen94

junwen94 commented Jul 8, 2026

Copy link
Copy Markdown
Collaborator

shall we also add mac/windows CI?

@sigilmakes

Copy link
Copy Markdown
Collaborator Author

Hmm. Personally I think that's not worthwhile at the moment? We're mainly targeting linux computers (HPC clusters or as a backend server), and it's all cross-platform python anyway atm. We could add it if we start needing to do per-platform handling

@junwen94
junwen94 merged commit 4c8b622 into main Jul 8, 2026
1 check passed
@sigilmakes
sigilmakes deleted the ci/github-actions branch July 8, 2026 10:53
@sigilmakes
sigilmakes restored the ci/github-actions branch July 8, 2026 10:53
@sigilmakes
sigilmakes deleted the ci/github-actions branch July 16, 2026 14: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.

2 participants