Page MenuHomePhabricator

[land-bot] Allow users to specify the branch/commit in order to determine the revision to land
ClosedPublic

Authored by jasonbcox on Oct 16 2020, 16:47.

Details

Summary

Specifying a branch does not work as expected. When working on stacked revisions, users
expect something like arc land-via-bot HEAD~ to land the revision associated with the commit
at HEAD~. This patch fixes that behavior.

Test Plan
git checkout <some-stacked-revision>
git cherry-pick <this-patch>
arc land-via-bot --preview
arc land-via-bot --preview HEAD~
arc land-via-bot --preview HEAD~2
arc land-via-bot --hold
arc land-via-bot --hold HEAD~
arc land-via-bot --hold HEAD~2

Diff Detail

Repository
rABC Bitcoin ABC
Branch
lvb-better-stack-support
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 13459
Build 26950: Build Difflint-circular-dependencies · build-without-wallet · build-diff · build-clang-tidy · build-clang
Build 26949: arc lint + arc unit