Pr workflow automation poc #74
Draft
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
|
Thanks for opening a pull request! If this is not a minor PR. Could you open an issue for this pull request on GitHub? https://github.com/apache/arrow/issues/new/choose Opening GitHub issues ahead of time contributes to the Openness of the Apache Arrow project. Then could you also rename the pull request title in the following format? or In the case of PARQUET issues on JIRA the title also supports: See also: |
raulcd
force-pushed
the
pr-workflow-automation-poc
branch
from
77bad41
to
8e95c04
Compare
fontim
reviewed
Feb 9, 2023
.github/workflows/pr_bot.yml
Outdated
| - opened | ||
| - converted_to_draft | ||
| - ready_for_review | ||
| pull_request_review: |
raulcd
force-pushed
the
pr-workflow-automation-poc
branch
2 times, most recently
from
413c899
to
a166340
Compare
raulcd
force-pushed
the
pr-workflow-automation-poc
branch
from
a166340
to
0d855f0
Compare
raulcd
force-pushed
the
pr-workflow-automation-poc
branch
from
8944685
to
fcf3305
Compare
raulcd
added a commit
to apache/arrow
that referenced
this pull request
Feb 28, 2023
### Rationale for this change As discussed on the [mailing list](https://lists.apache.org/thread/1rhsd8ovy4bfr8hcdohn0vh65frw0ggk) is quite difficult to understand the current state of a PR whether it requires further review, it has gone stale or new changes have been added. This allows us to have a set of labels based on the state of the PR. ```mermaid flowchart TD A([New PR]):::creator A -- by non-committer --> B[Awaiting review]:::anyone A -- by committer --> C[Awaiting commiter review]:::committer B & C -- new review by\nanother non-committer --> C C & B & E -- new committer review\nrequests changes --> D[Awaiting changes]:::creator D -- changes by creator --> E[Awaiting change review]:::committer C & E & B -- new committer review\napproves ---> F[Awaiting merge]:::committer classDef creator stroke:#CC0; classDef anyone stroke:#00C; classDef committer stroke:#0C0; classDef triager stroke:#C0C; linkStyle 0,1,7 stroke:#CC0,color:auto; linkStyle 2,3 stroke:#00C,color:auto; linkStyle 4,5,6,8,9,10 stroke:#0C0,color:auto; ``` ### What changes are included in this PR? New workflow to trigger archery bot on the required actions. New PR Workflow bot implementation on archery that manages the GitHub events and state. New fixtures and tests. ### Are these changes tested? There are unit tests and has been tested on my fork. Some transition examples on this PR: raulcd#74 * Closes: #33977 Authored-by: Raúl Cumplido <raulcumplido@gmail.com> Signed-off-by: Raúl Cumplido <raulcumplido@gmail.com>
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Rationale for this change
What changes are included in this PR?
Are these changes tested?
Are there any user-facing changes?