Skip to content

security: define release-path code owners - #498

Merged
codeforester merged 2 commits into
mainfrom
security/484-20261028-release-controls
Oct 28, 2026
Merged

codeforester merged 2 commits into
mainfrom
security/484-20261028-release-controls

Conversation

@codeforester

Copy link
Copy Markdown
Contributor

Summary

  • add CODEOWNERS coverage for release workflows, VERSION, packaging metadata, locks, and release validators
  • document the independent-review and emergency-recovery controls for production releases
  • prepare the repository for the matching protected-main and v* tag rulesets

Validation

  • UV_CACHE_DIR=/private/tmp/base-cli-uv-cache uv run --extra dev pytest -q tests/test_security_documentation.py tests/test_validate_docs.py
  • Ruff and git diff --check pass

After this merges, apply the repository ruleset changes from #484: one approval, code-owner review, last-push approval, and immutable v* tags.

Relates to #484.

Comment thread docs/releasing.md Outdated
Comment thread .github/CODEOWNERS
Comment thread .github/CODEOWNERS Outdated
Comment thread .github/CODEOWNERS Outdated
Comment thread docs/releasing.md
@codeforester

Copy link
Copy Markdown
Contributor Author

Implemented the review fixes in commit fbf7a946.

  • Release documentation now distinguishes required controls from controls verified as active in live repository settings; it no longer asserts that tag immutability or ruleset enforcement is already enabled.
  • CODEOWNERS now protects itself and uses explicit release-sensitive files instead of broad /pyproject.toml, /requirements/, /scripts/validate_*, and /scripts/verify_* patterns.

@codeforester
codeforester force-pushed the security/484-20261028-release-controls branch from fbf7a94 to 53cab3d Compare October 28, 2026 15:01
@codeforester
codeforester merged commit be62cfc into main Oct 28, 2026
118 of 120 checks passed
@codeforester
codeforester deleted the security/484-20261028-release-controls branch October 28, 2026 15:09
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