Page MenuHomePhabricator

Fix "left"/"right" changeset ID selection for synthetic deletions
ClosedPublic

Authored by epriestley on May 17 2017, 3:27 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, May 4, 6:54 PM
Unknown Object (File)
Wed, May 1, 12:21 AM
Unknown Object (File)
Sat, Apr 13, 8:36 PM
Unknown Object (File)
Sat, Apr 13, 3:36 PM
Unknown Object (File)
Sat, Apr 13, 3:36 PM
Unknown Object (File)
Sat, Apr 13, 11:05 AM
Unknown Object (File)
Sat, Apr 13, 9:07 AM
Unknown Object (File)
Sat, Apr 13, 9:07 AM
Subscribers
None

Details

Summary

Fixes T8323. See that task for a description.

We were using nonempty(), but that rule doesn't cover synthetic deletions (file present in an earlier diff, but no longer present in the later diff).

Test Plan

Followed the steps in T8323, got a clean comment.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

This revision is now accepted and ready to land.May 17 2017, 3:31 PM
This revision was automatically updated to reflect the committed changes.