This clarifies the error message in less-common cases such as
when a revision has already been landed. Prior to this patch,
it appears as if the revision was never reviewed at all. It's best
that we provide the review status to the user so they can make
a judgement of what the root cause is.
Details
Details
- Reviewers
Fabien - Group Reviewers
Restricted Project - Commits
- rABC8227e3687e9b: [land-bot] Clarify review status error message
CONDUIT_TOKEN=<token> ./land-patch.sh -r D6379
Outputs:
Error: Revision 'D6379' is not accepted. Review status is: "published"
Diff Detail
Diff Detail
- Repository
- rABC Bitcoin ABC
- Branch
- landbot-rev-status
- Lint
Lint Passed - Unit
No Test Coverage - Build Status
Buildable 11229 Build 21167: Default Diff Build & Tests Build 21166: Build without Wallet Build 21165: Build with clang-10 Build 21164: arc lint + arc unit