Skip to content

Standardize justfile targets - #360

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

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

Conversation

@blink1073

@blink1073 blink1073 commented Aug 11, 2026

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. Recipes also did not control their own environment, so running one target could leave dependencies in a state the next target did not expect. Each recipe now syncs the exact dependency groups it needs, which makes any order of commands safe.

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
  • Every recipe syncs the dependency groups it needs before doing its work
  • 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 13 source files
  • just test passes, 174 tests
  • Ran just test followed by just lint to confirm the second command re-syncs its group instead of failing on a pruned environment

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:38
@blink1073
blink1073 enabled auto-merge (squash) August 11, 2026 00:38
@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 92.93%. Comparing base (1c456f3) to head (714d541).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main     #360   +/-   ##
=======================================
  Coverage   92.93%   92.93%           
=======================================
  Files           6        6           
  Lines         439      439           
=======================================
  Hits          408      408           
  Misses         31       31           
Flag Coverage Δ
linux 92.71% <ø> (ø)
windows 92.93% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

🚀 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 29b0477 into Calysto:main Aug 11, 2026
30 of 31 checks passed
@blink1073
blink1073 deleted the standardize-justfile-targets branch August 11, 2026 00:44
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