Page MenuHomePhabricator

[land-bot] Move apply-revision to patch recipes
AbandonedPublic

Authored by jasonbcox on Sep 21 2020, 23:04.

Details

Reviewers
deadalnix
Group Reviewers
Restricted Project
Summary

This allows us to use automated-commits.sh as our land bot entry point.

Alternative to D7479

Depends on D7490

Test Plan
CONDUIT_TOKEN=<token> COMMIT_TYPE=dummy ./automated-commits.sh -p HEAD ./patch-recipes/apply-revision.sh Dxxxx

Diff Detail

Repository
rABC Bitcoin ABC
Branch
apply-rev-recipe
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 12832
Build 25734: Build Diffbuild-diff · build-clang-tidy · build-clang-10 · build-without-wallet
Build 25733: arc lint + arc unit

Event Timeline

deadalnix requested changes to this revision.Sep 21 2020, 23:29
deadalnix added a subscriber: deadalnix.

How does putting this script in another folder allow to do anything with the landbot?

There is clearly some piece of the puzzle that is not explained.

This revision now requires changes to proceed.Sep 21 2020, 23:29

Clearly this was possible with D7490 already and nothing really needs to be done. It can be argued that moving it to patch-recipes isn't even the right place anyway. Sorry for the noise.