Page MenuHomePhabricator

Add "Is merge commit?" to commit fields in Herald
ClosedPublic

Authored by epriestley on Dec 5 2015, 8:58 PM.
Tags
None
Referenced Files
F15514151: D14685.id35517.diff
Fri, Apr 18, 12:54 AM
F15510893: D14685.id.diff
Wed, Apr 16, 9:51 PM
F15508780: D14685.id35516.diff
Wed, Apr 16, 6:30 AM
F15505908: D14685.diff
Tue, Apr 15, 6:53 AM
F15471081: D14685.id35517.diff
Sat, Apr 5, 3:08 AM
F15462812: D14685.diff
Tue, Apr 1, 5:55 PM
F15395694: D14685.id.diff
Mar 16 2025, 9:00 AM
F15381077: D14685.diff
Mar 14 2025, 5:45 AM
Subscribers
Tokens
"Like" token, awarded by avivey.

Details

Summary

Fixes T5788. We already have this as a pre-commit field, add it as a post-commit field too.

Test Plan

Ran this rule on a merge commit. Also ran it on a non-merge commit. Both got the correct value.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Add "Is merge commit?" to commit actions in Herald.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
avivey added a reviewer: avivey.
avivey added a subscriber: avivey.

The title should probably be "...rule in herald"?

This revision is now accepted and ready to land.Dec 5 2015, 9:07 PM
epriestley retitled this revision from Add "Is merge commit?" to commit actions in Herald to Add "Is merge commit?" to commit fields in Herald.
epriestley edited edge metadata.

Ah, yes.

chad edited edge metadata.
This revision was automatically updated to reflect the committed changes.