forked from facebook/astryx
-
Notifications
You must be signed in to change notification settings - Fork 0
Review 5171
Cindy Zhang edited this page Aug 23, 2026
·
1 revision
nynexman4464 · merged · view on GitHub
Verdict: approve
These records predate the Review Presentation format for the most part, so the problem/solution/API/theme-target/breaking slots are only filled where the review itself carried them. Reviews from 2026-08-23 on use the full template.
Thanks — ran it against main: 8 hits, 3 real. Nit before merge:
charts/formatters.tsis exempt wholesale, so the file the bug was in gets no enforcement.[Reviewed by Robohands]