Skip to content

Standardize justfile targets - #474

Merged
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:standardize-justfile-targets
Aug 11, 2026
Merged

Standardize justfile targets#474
blink1073 merged 1 commit into
Calysto:mainfrom
blink1073:standardize-justfile-targets

Conversation

@blink1073

Copy link
Copy Markdown
Contributor

References

None

Description

The five developer commands now mean the same thing in every Calysto repo: install, test, cover, typing, and lint.

Two things were confusing before. lint ran a hand-picked subset of pre-commit hooks while a separate pre-commit recipe ran all of them, so neither name described what it did. The lint recipes also did not control their own environment. Because just test syncs to the test group and prunes everything else, running it before just lint left no pre-commit in the environment. Each recipe now syncs the groups it needs, so any order of commands works.

Changes

  • Renamed the run-all-hooks recipe from pre-commit to lint
  • Added lint-all for hooks that only run at the manual stage, such as markdown-link-check
  • The lint recipes now sync the dev group before running, matching what the other recipes already did
  • Updated the contributing guide to match

Backwards-incompatible changes

just pre-commit no longer exists. Use just lint, or just lint-all to include the manual-stage hooks.

Testing

  • just lint passes
  • just typing passes with no issues in 113 source files
  • just test passes, 489 tests with 68 skipped

AI usage

  • Some or all of the content of this PR was generated by AI.
  • The human author has carefully reviewed this PR and run this code.
  • AI tools and models used: Claude Code (Opus 5)

Every recipe syncs the exact dependency groups it needs with
`poetry sync`, so moving between targets can't leave a stale
environment behind.

Rename the all-hooks pre-commit recipe to `lint` and add `lint-all`
for manual-stage hooks.
@blink1073
blink1073 marked this pull request as ready for review August 11, 2026 00:39
@blink1073
blink1073 enabled auto-merge (squash) August 11, 2026 00:39
@codecov

codecov Bot commented Aug 11, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 93.64%. Comparing base (11a067b) to head (f033ae8).
⚠️ Report is 7 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##             main     #474      +/-   ##
==========================================
+ Coverage   93.54%   93.64%   +0.10%     
==========================================
  Files          51       51              
  Lines        2974     2960      -14     
  Branches      416      410       -6     
==========================================
- Hits         2782     2772      -10     
+ Misses        131      128       -3     
+ Partials       61       60       -1     

see 8 files with indirect coverage changes

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@blink1073
blink1073 disabled auto-merge August 11, 2026 00:43
@blink1073
blink1073 merged commit 93d58fe into Calysto:main Aug 11, 2026
25 of 26 checks passed
@blink1073
blink1073 deleted the standardize-justfile-targets branch August 11, 2026 00:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant