Skip to content

Format workflow files#10

Merged
amaksimo merged 1 commit into
mainfrom
dev/frankcom/format-workflows
Jan 28, 2026
Merged

Format workflow files#10
amaksimo merged 1 commit into
mainfrom
dev/frankcom/format-workflows

Conversation

@danielfrankcom

Copy link
Copy Markdown
Contributor

This PR applies auto-formatting to workflows, in preparation for an upcoming change to the pre-commit checks.

We were previously checking this in some of the subprojects, so I am combining those checks to run at the root level. I've split this change out to make the final diff smaller and isolate formating-only changes.

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@amaksimo amaksimo merged commit 76c1ef5 into main Jan 28, 2026
58 checks passed
@amaksimo amaksimo deleted the dev/frankcom/format-workflows branch January 28, 2026 22:40
danielfrankcom added a commit that referenced this pull request Jan 28, 2026
This PR consolidates development tooling and CI workflows from the
individual project repos into a monorepo-compatible format at the top
level.

### Changes

Pre-commit hooks (`.pre-commit-config.yaml`)
- Added root-level pre-commit config covering all projects
- Python: `ruff` (lint + format), `pyright` (sqlalchemy, tortoise-orm
only)
- Java: Google Java formatter
- YAML: prettier
- GitHub Actions: actionlint
- Secrets: gitleaks
- Removed per-project `.pre-commit-config.yaml` files (sqlalchemy,
tortoise-orm)

CI workflows
- Added pre-commit workflow to run pre-commit checks
- Removed redundant lint jobs from Python CI workflows (now handled by
pre-commit workflow)
- Added build step to sqlalchemy and tortoise-orm CI workflows
- Added uv lock --check to all Python CI workflows to catch out-of-sync
lock files

Consolidated files
- Moved `LICENSE_HEADER.txt` to repo root, removed per-project copies
- Moved `.mise.toml` to repo root, removed per-project copies
- Added pre-commit setup instructions to `CONTRIBUTING.md`

This PR depends on #10, #11, and #12 to make the checks pass. Once those
are merged I will mark this as ready for review. Until then, the checks
will fail (as they are designed to do).

By submitting this pull request, I confirm that you can use, modify,
copy, and redistribute this contribution, under the terms of your
choice.
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