RFR: 1949: Handle maintainer approval from pull request [v2]
Zhao Song
zsong at openjdk.org
Thu Aug 17 03:03:28 UTC 2023
> As Erik said in the issue, "In SKARA-1199, we added the optional ability for a PR to block integration until a certain approval label appears on all the associated bugs. To further improve the workflow, we would like to add a couple of PR commands that will make it possible to perform the whole maintainer approval workflow through the PR"
>
> Therefore, in this patch, two pull request commands(`/approval` and `/approve`) are introduced.
>
> 1. Command `/approval`
> Usage: `/approval [<id>] (request|cancel) [<text>]`
> Examples: `/approval request my reason`, `/approval cancel`,
> `/approval JDK-123 request my reason`, `/approval 123 cancel`
>
> Only the author of the pull request is allowed to issue this command. If the user requested approval, the text will be posted to the bug. The user will be able to update the text by requesting the approval again. Once maintainer processed the request(approved or rejected), the user will not be able to request or cancel the approval.
>
> If there is only one issue associated with the pr, the user will not need to specify the issue id. If there are more than one issues associated with the pr, the user should specify the issue id in the command.
>
> 2. Command `/approve`
> Usage: `/approve [<id>] (yes|no)`
> Examples: `/approve yes`, `/approve no`, `/approve JDK-123 yes`, `/approve 123 no`
>
> Only maintainers of the repo are allowed to issue this command. Maintainers can not approve or reject the approval if the request doesn't exist.
>
> If the user doesn't specify the issue id in the command, then all the approval request in issues associated with pull request will be handled.
Zhao Song has updated the pull request incrementally with 21 additional commits since the last revision:
- update
- fix test
- reviewComment
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/CheckRun.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApproveCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApprovalCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApproveCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApproveCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApprovalCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- Update bots/pr/src/main/java/org/openjdk/skara/bots/pr/ApprovalCommand.java
Co-authored-by: Erik Joelsson <37597443+erikj79 at users.noreply.github.com>
- ... and 11 more: https://git.openjdk.org/skara/compare/aab28673...e488acc6
-------------
Changes:
- all: https://git.openjdk.org/skara/pull/1544/files
- new: https://git.openjdk.org/skara/pull/1544/files/aab28673..e488acc6
Webrevs:
- full: https://webrevs.openjdk.org/?repo=skara&pr=1544&range=01
- incr: https://webrevs.openjdk.org/?repo=skara&pr=1544&range=00-01
Stats: 261 lines in 13 files changed: 184 ins; 11 del; 66 mod
Patch: https://git.openjdk.org/skara/pull/1544.diff
Fetch: git fetch https://git.openjdk.org/skara.git pull/1544/head:pull/1544
PR: https://git.openjdk.org/skara/pull/1544
More information about the skara-dev
mailing list