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)
Nov 22 2024, 9:04 AM
Unknown Object (File)
Nov 19 2024, 4:38 AM
Unknown Object (File)
Nov 15 2024, 3:57 AM
Unknown Object (File)
Nov 10 2024, 11:24 PM
Unknown Object (File)
Nov 9 2024, 12:02 PM
Unknown Object (File)
Nov 7 2024, 6:12 AM
Unknown Object (File)
Nov 2 2024, 10:33 PM
Unknown Object (File)
Oct 24 2024, 5:41 AM
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.