mirror of
https://github.com/kevinveenbirkenbach/computer-playbook.git
synced 2026-11-08 03:38:23 +00:00
fix(ci): make 🧩 Subset resolver comply with repo lint rules
The merged subset feature failed test-lint: a .py under scripts/, a local parents[N] PROJECT_ROOT, a direct yaml.safe_load, an uncached read, and narration comments. - move scripts/meta/resolve/pr/subset_roles.py -> cli.meta.ci.subset_roles (scripts/ is .sh-only); dispatcher invokes it via python -m - parse via utils.cache.yaml.load_yaml_str; import utils.PROJECT_ROOT - test invokes the module via -m against real roles; nocheck the tempfile read - strip narration comments; docs document HOW not WHY Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
This commit is contained in:
1 parent
6ad17c07de
commit
0bcd63b658
10 files changed
+66
-75
No files matched your search
@@ -8,9 +8,9 @@
|
||||
# Inputs via env:
|
||||
# SUBSET_LABELED "true" when the PR carries the '🧩 Subset' label.
|
||||
# When set, the explicit role list declared in the PR
|
||||
# body wins, resolved by
|
||||
# scripts/meta/resolve/pr/subset_roles.py (strict: it
|
||||
# fails the run on invalid/empty/unknown roles).
|
||||
# body wins, resolved by the cli.meta.ci.subset_roles
|
||||
# module (strict: it fails the run on
|
||||
# invalid/empty/unknown roles).
|
||||
# Otherwise the diff-derived resolver runs unchanged.
|
||||
# PR_BODY PR body markdown (only read on the subset path).
|
||||
#
|
||||
@@ -24,7 +24,7 @@ set -euo pipefail
|
||||
|
||||
if [[ "${SUBSET_LABELED:-}" == "true" ]]; then
|
||||
echo "🧩 Subset label present: restricting CI to the roles listed in the PR body."
|
||||
exec python scripts/meta/resolve/pr/subset_roles.py
|
||||
exec python -m cli.meta.ci.subset_roles
|
||||
fi
|
||||
|
||||
resolved="$(./scripts/meta/resolve/diff/affected_roles.sh)"
|
||||
|
||||
Reference in new issue
Block a user