Page MenuHomePhabricator

Status Closed should also concieved as Accepted
AbandonedPublic

Authored by dragoon20 on Jun 27 2016, 8:48 AM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 4:38 AM
Unknown Object (File)
Fri, Nov 15, 3:57 AM
Unknown Object (File)
Sun, Nov 10, 11:24 PM
Unknown Object (File)
Sat, Nov 9, 12:02 PM
Unknown Object (File)
Thu, Nov 7, 6:12 AM
Unknown Object (File)
Sat, Nov 2, 10:33 PM
Unknown Object (File)
Thu, Oct 24, 5:41 AM
Unknown Object (File)
Oct 19 2024, 4:07 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

This is related to a scenarion in https://secure.phabricator.com/T4754.

  • arc feature
  • ... do work ...
  • arc diff
  • (Diff accepted)
  • arc land
  • (Herald rule checks status == accepted, allows commit)
  • (Commit worker detects diff and closes it)
  • git checkout lts
  • git cherry-pick master
  • git push ....
  • (Herald rule sees status == closed, rejects push) *(this should be allowed)*
Test Plan

Herald rule for differential accepted should return true when diff is closed

Diff Detail

Repository
rP Phabricator
Branch
master
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 12805
Build 16309: arc lint + arc unit

Event Timeline

dragoon20 retitled this revision from to Status Closed should also concieved as Accepted.
dragoon20 updated this object.
dragoon20 edited the test plan for this revision. (Show Details)
dragoon20 added a reviewer: epriestley.
epriestley edited edge metadata.

See T9838 / D15085.

This revision now requires changes to proceed.Jun 27 2016, 12:00 PM

I see, thanks for the comment.
Hope it will land soon.