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)
Wed, Apr 24, 10:43 PM
Unknown Object (File)
Thu, Apr 11, 7:38 AM
Unknown Object (File)
Mar 23 2024, 10:50 PM
Unknown Object (File)
Feb 24 2024, 12:42 PM
Unknown Object (File)
Feb 13 2024, 5:52 PM
Unknown Object (File)
Jan 16 2024, 10:12 AM
Unknown Object (File)
Jan 4 2024, 3:02 PM
Unknown Object (File)
Nov 30 2023, 5:20 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.