Page MenuHomePhabricator

Herald - fix change type bug
ClosedPublic

Authored by btrahan on Apr 8 2014, 6:54 PM.
Tags
None
Referenced Files
F15323519: D8724.id20687.diff
Fri, Mar 7, 12:38 AM
Unknown Object (File)
Tue, Feb 25, 4:37 PM
Unknown Object (File)
Tue, Feb 25, 4:37 PM
Unknown Object (File)
Tue, Feb 25, 3:01 PM
Unknown Object (File)
Mon, Feb 17, 1:48 AM
Unknown Object (File)
Thu, Feb 13, 11:59 PM
Unknown Object (File)
Jan 31 2025, 4:18 AM
Unknown Object (File)
Jan 28 2025, 6:43 PM
Subscribers

Details

Summary

wasn't working due to some type issues. Fixes T4756. I also made it display nicer while I was debugging this.

Test Plan

created a herald rule to block changes that added refs. git tag -a "test" -m "test test"; git push origin test got me blocked!

Diff Detail

Repository
rP Phabricator
Branch
T4756
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

btrahan retitled this revision from to Herald - fix change type bug.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
This revision is now accepted and ready to land.Apr 8 2014, 6:57 PM
btrahan updated this revision to Diff 20689.

Closed by commit rPd5ded805b2d3 (authored by @btrahan).