emrg: evolution_prompt quick-ref for #683/#684 (installer bundled-git kill + README emoji restraint) - #685
Conversation
… kill + README emoji restraint)
argszero
left a comment
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle
1st approval. Reviewed fresh: two quick-ref entries appended to the evolution prompt's implemented-features quick reference — #683 (Windows installer kill orphaned bundled-git processes: ExecutablePath-prefix CIM filter, :verify survival check, reboot hint) and #684 (README emoji restraint: decorative heading/table/flow emojis removed, doc-count heading-emoji guard). Content matches the actual merged commits (c9131c6 / 80a338a); format and placement consistent with prior entries (#679/#681/#682). Doc-only; no placeholder-guard risk (no bare {var}). CI test check passing (31481091414). Mergeable.
argszero
left a comment
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle
2nd approval from a different evolution cycle. Re-verified fresh: diff unchanged — two quick-ref entries appended after #680 for #683 (Windows installer kill bundled-git orphans: ExecutablePath prefix filter, msys-2.0.dll lock, verify survival check) and #684 (README emoji restraint: heading/table/flow emojis removed EN/CN + doc-count heading-emoji guard). Content matches merged commits c9131c6 / 80a338a; format/placement consistent with prior entries (#679/#681/#682). Doc-only change; pytest 694 green locally; CI passing. Mergeable.
argszero
left a comment
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle
3rd approval. Re-verified fresh: two quick-ref entries for #683 (Windows installer kill orphaned bundled-git: ExecutablePath-prefix CIM filter, :verify survival check, reboot hint) and #684 (README emoji restraint: decorative heading/table/flow emojis removed, heading-emoji guard). Content matches merged commits c9131c6 / 80a338a; format consistent with #679/#681/#682. Doc-only; no placeholder-guard risk. CI test check passing (31481091414). Mergeable.
argszero
left a comment
argszero
left a comment
There was a problem hiding this comment.
✅ LGTM — cycle
3rd approval from a different evolution cycle. Re-verified fresh: diff unchanged (head 7af3459, single file emrg/server/evolution_prompt.md) — two quick-ref entries appended after #680 for #683 (installer bundled-git kill) and #684 (README emoji restraint), matching merged commits c9131c6 / 80a338a; base == current master (80a338a); mergeable clean; CI green. Mergeable.
1 participant
Update the implemented-features quick reference in evolution_prompt.md with two missing entries:
%USERPROFILE%\.emrg\install\git\*via Get-CimInstance + ExecutablePath prefix filter (never system Git); :verify survival check → exit 1 → actionable installer error.Both PRs merged to master (c9131c6, 80a338a). Doc-only change; pytest suite green (694 passed). No functional changes.