Skip to content

docs: Add pre-commit CI#300

Merged
ericholscher merged 1 commit intomainfrom
sync-agents-pre-commit
May 1, 2026
Merged

docs: Add pre-commit CI#300
ericholscher merged 1 commit intomainfrom
sync-agents-pre-commit

Conversation

@ericholscher
Copy link
Copy Markdown
Member

@ericholscher ericholscher commented Apr 30, 2026

Makes the repo's standard pre-commit command work locally by adding the expected .pre-commit-config.yaml symlink and keeping the hook-applied formatting updates.

Adds a basic GitHub Actions workflow that runs pre-commit across the backend/shared files on pushes to main and pull requests.

Keeps StylesPath = docs/vale/ for compatibility with repos that symlink .vale.ini from common, and adds docs/vale -> ../vale inside this repo so the same config works here too.

The AGENTS sync script remains in #299.

Validation:

  • uvx --python 3.12 pre-commit run --all-files passed.
  • uvx --python 3.12 pre-commit run --files .github/workflows/pre-commit.yml .pre-commit-config.yaml passed.

Generated by the AI agent in use.

@ericholscher ericholscher force-pushed the sync-agents-pre-commit branch from 0cfae49 to d3353f7 Compare April 30, 2026 18:57
@ericholscher ericholscher changed the title docs: Add AGENTS sync script and pre-commit CI docs: Add pre-commit CI Apr 30, 2026
@ericholscher ericholscher force-pushed the sync-agents-pre-commit branch from d3353f7 to 01297fe Compare April 30, 2026 19:13
@ericholscher ericholscher marked this pull request as ready for review April 30, 2026 19:13
@ericholscher ericholscher requested a review from a team as a code owner April 30, 2026 19:13
@ericholscher ericholscher requested a review from stsewd April 30, 2026 19:13
@ericholscher ericholscher merged commit 31c6ccc into main May 1, 2026
1 check passed
@ericholscher ericholscher deleted the sync-agents-pre-commit branch May 1, 2026 11:54
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