Skip to content

build(make): include Bandit and workflow lints in check - #851

Merged
yeongseon merged 2 commits into
mainfrom
build/811-check-contract
Oct 26, 2026
Merged

yeongseon merged 2 commits into
mainfrom
build/811-check-contract

Conversation

@yeongseon

Copy link
Copy Markdown
Owner

Summary

Complete the remaining make check contract from #811 without duplicating offline gates.

Changes

  • check runs lint (including format and mypy), the existing full Bandit scope, and workflow lints.
  • check-all calls check and test, retaining every prior gate exactly once.
  • Contract tests cover both aggregates and contributor docs describe the deliberate command change.

Validation

  • Regression first: check omitted all four expected security/workflow commands; check-all already passed.
  • make check-all passed: 1651 passed, 30 skipped, 22 deselected, 5 warnings in 27.25s; coverage 95.25%.
  • make docs and make build passed.
  • GitHub CI pending.

Related Issues

Refs #811

This focused follow-up addresses the static-check acceptance item; it does not re-certify every historical item in the closed umbrella issue.

Merge authorized by the maintainer’s explicit instruction.

@yeongseon
yeongseon merged commit 2b0d081 into main Oct 26, 2026
29 checks passed
@yeongseon
yeongseon deleted the build/811-check-contract branch October 26, 2026 14:12
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