Skip to content

Commit 0bcbb17

Browse files
committed
Point breeze verify at the prek hooks a default install does not run
Dropping the static-check rows also hid mypy-providers and migration-round-trip, which CI runs but a default prek install never triggers (pre-push and manual stage), so a migration change got no hint until CI failed. The CLI tests also ran real git against the developer's checkout and global config. Generated-by: Claude Opus 5
1 parent 13b7f31 commit 0bcbb17

6 files changed

Lines changed: 138 additions & 45 deletions

File tree

‎dev/breeze/doc/03_developer_tasks.rst‎

Lines changed: 6 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -342,8 +342,12 @@ Each row says what kind of check it is, whether it runs on the host or needs Doc
342342
CI image (``breeze``), and the exact command. Jobs CI runs on every PR regardless of the change
343343
(breeze's own unit tests, the shared distributions) only show up with ``--full``. The translation
344344
check is never listed: CI runs it with ``|| true``, so it cannot fail a PR.
345-
Static checks are not listed either, including the ``mypy-providers`` and ``migration-round-trip``
346-
jobs: ``prek`` already picks the hooks to run for the changed files, so run it as usual.
345+
Static checks are not listed either: ``prek`` already picks the hooks to run for the changed files,
346+
so run it as usual. The exception is the ``mypy-providers`` and ``migration-round-trip`` hooks,
347+
which CI runs but a default ``prek install`` does not (they are ``pre-push`` or ``manual`` stage
348+
hooks). When the change triggers them, ``breeze verify`` prints the
349+
``prek run --stage manual <hook> --from-ref <base>`` command to run each one, and ``--json``
350+
lists them under ``manual_prek_hooks``.
347351
Use ``--json`` for machine-readable output with the same fields.
348352

349353
When a change touches CI tooling or dependency files, selective checks make CI run the full suite.

0 commit comments

Comments
 (0)