Document the experimental Rush reporter demo - #5998
Document the experimental Rush reporter demo#5998Sean Larkin (TheLarkInn) wants to merge 1 commit into
Conversation
|
Independent review completed against #5983, #5858, the RFC, and the #5987/#5989/#5993/#5996/#5997 implementation slices. Fixed in
Executed the clean-checkout install/build/demo flow, the demo again under hostile inherited reporter controls, pseudo-TTY No review threads existed to resolve. #5993 behavior remains explicitly labeled as an unmerged prerequisite, automatic AI selection remains gated by open #5981, and |
|
Combined deep review of current head No critical, high, or medium actionable findings. LOW Recheck the log cleanup and telemetry privacy guarantees after the upstream temp-folder and telemetry projection fixes land. Scope the wording if either guarantee remains conditional. |
4082b91 to
5543e9b
Compare
|
Rebased the three docs/demo commits onto exact #5997 head Fixed docs findings: normalized Deferred low: third-party Validation: expanded demo matrix, pseudo-TTY default, reporter 317, rush-lib 788, Rush frontend 71, |
|
Final combined review of Dismissible low follow-ons:
All other documented controls, compatibility, privacy, watch, temp, purge, AI, JSON, rollback, and Rush 6 gates match the reviewed implementation. This head is low-only. |
5543e9b to
71cf5cc
Compare
71cf5cc to
26fa428
Compare
26fa428 to
762bc89
Compare
762bc89 to
f06cad9
Compare
f06cad9 to
e891812
Compare
e175479 to
bbae74b
Compare
bbae74b to
24b5ca6
Compare
18e149a to
da6d17a
Compare
da6d17a to
3ffde88
Compare
3ffde88 to
ed255e8
Compare
ed255e8 to
a0e3654
Compare
a0e3654 to
ecf0978
Compare
Sean Larkin (TheLarkInn)
left a comment
There was a problem hiding this comment.
R10 acceptance review against #5983: exact documented commands must run from the pinned checkout.
ecf0978 to
c677d60
Compare
Address #5998 review 5154326282. Document the shell-wide preview selection and cleanup for individual examples while preserving the already-seeded integration driver. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10
Preserve the exact published documentation and self-checking driver, including explicit built-version selection and inherited reporter-control isolation. Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com> Copilot-Session: 4747f826-8c83-495d-80df-3d1168662e10
c677d60 to
0af5771
Compare
Part of #5983
Stack
97f807ed6549387b6aab2eccec197123196b725dThis PR is based directly on the reviewed #5997 tip. It does not include #5990, #5993, or the parallel R7/R8 branches. Auto-merge remains disabled while stack prerequisites are open.
Documentation
--reporter, repeatable--output,--log-level, verbosity aliases, command-specific--json, precedence, conflicts, and stream ownership.RUSH_REPORTER=legacy, no automatic AI selection, and the Rush 6 safety boundary.RUSH_TEMP_FOLDER, purge cleanup, fallback permissions, and privacy.Validation
@microsoft/rushdefaultreporterrush checkrush change --verify --no-fetch70a28dd51353fc7579c7483c97e327c82eff6f98andbe34125a375964aad356cf38e1da29ad2409d57bSafety boundary
This PR does not change Rush 5 defaults and does not enable environment-based reporter or AI auto-selection. Rush 6 behavior is described only as planned and gated on #5990, #5993, #5981, and #5982.