Skip to content

ci: run checks without fixing files - #94

Merged
galuszkm merged 2 commits into
mainfrom
ci/verify-no-fix
Nov 1, 2026
Merged

galuszkm merged 2 commits into
mainfrom
ci/verify-no-fix

Conversation

@galuszkm

@galuszkm galuszkm commented Nov 1, 2026

Copy link
Copy Markdown
Member

Description

CI now only checks the code. It never changes files.

  • New just verify task: runs the same checks as just check (format, import order, lint, types, security) without fixing anything. just check still fixes files for local use.
  • CI runs just verify: before, CI used the fixing tasks, so badly formatted code could be fixed on the CI machine and still pass. Now it fails and the author has to fix it.
  • Docs: CONTRIBUTING.md and tasks/README.md describe the new tasks.
  • Lockfile: the package version in uv.lock is updated to 0.12.0 to match the release.

Related Issues

None

Type of Change

  • Other (please describe): CI and developer tooling

YAML / API Impact

None. No change to the config schema or the Python API.

Testing

How have you tested the change?

  • I ran uv run just check (lint + type check)
  • I ran uv run just test for overall testing
  • I added or updated tests that prove my fix is effective or my feature works
  • I verified existing examples in examples/ still work

Checklist

  • I have read the CONTRIBUTING document
  • I have updated the documentation accordingly
  • My changes generate no new warnings
  • Any dependent changes have been merged and published

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

- Add just verify, the same checks as just check that never change a file
- CI runs just verify instead of the fixing check tasks
@galuszkm
galuszkm merged commit 5160354 into main Nov 1, 2026
9 checks passed
@galuszkm
galuszkm deleted the ci/verify-no-fix branch November 1, 2026 12:40
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