Repository navigation
make setup-pstack medium and small set every role - #44
Conversation
medium is flash, small is flash_lite. the readme maps roles to tiers. Co-authored-by: Kaitian Xie <caelaxie@users.noreply.github.com>
caelaxie
left a comment
There was a problem hiding this comment.
verdict: changes requested (blocked on proof only).
content passes. the diff is exactly the smallest fix from #41: medium sets every role to flash, small sets every role to flash_lite, aliases stay put, and the readme drops the detector line and names the tiers. two files, +2/-2, nothing else touched. layout fine, Closes #41 present, not a draft.
missing: harness proof. skills/setup-pstack/SKILL.md is plugin content, so it needs agy plugin install + agy plugin list run against head 6123b251d33b6cca39f5c1db6d04255d3c131b50, showing the installed setup-pstack/SKILL.md carries the new step 3b text. post it as a pr comment or in the body. Validate plugin is still running; it needs to go green too.
merge follows once both land.
|
harness on head |
caelaxie
left a comment
There was a problem hiding this comment.
pass, merging.
content passed last round (smallest fix from #41). proof is now on the exact head: agy 1.4.7 validate + install + list, installed setup-pstack cmp matches the head blob, Validate plugin green. Closes #41 set.
Closes #41
mediumsets every role toflash.smallsets every role toflash_lite.inherit,inherit-parent, andautostay put. the readme maps each role to a tier.head
6123b251d33b6cca39f5c1db6d04255d3c131b50harness on that sha. installed
skills/setup-pstack/SKILL.mdhas the step 3b text.cmpvs the head blob exits 0.Validate pluginis green: https://github.com/pstack-ports/agy-pstack/actions/runs/40261539266/job/121374786336