Details
Details
Diff Detail
Diff Detail
Unit Tests
Unit Tests
Event Timeline
Comment Actions
This patch does not meet basic standards.
- It has no explanation of intent.
- It has no tests.
- The code change itself seems nonsensical.
Phabricator doesn't give a way to hard reject, so marking as Request Changes and removing myself as reviewer.