Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

[CI] Ping Reviewers Opt-In #9983

Open
driazati opened this issue Jan 19, 2022 · 0 comments
Open

[CI] Ping Reviewers Opt-In #9983

driazati opened this issue Jan 19, 2022 · 0 comments
Labels
dev:tools type:rfc-tracking RFC progress tracking. Ref: https://github.com/apache/tvm-rfcs

Comments

@driazati
Copy link
Member

driazati commented Jan 19, 2022 •
edited by github-actions bot
Loading

This is the list of people whose PRs are opted-in to the features in #9973 (the code in that PR reads directly from this issue). Once that bot has been validated it'll be opened up to everyone:

Opted in users:
cc @Mousius @areusch @driazati @gigiblender
@driazati driazati added the type:rfc-tracking RFC progress tracking. Ref: https://github.com/apache/tvm-rfcs label Jan 19, 2022
@driazati driazati mentioned this issue Jan 21, 2022
Merged
@driazati driazati mentioned this issue Feb 10, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Feb 10, 2022
@driazati
[ci] Check more events before pinging reviewers
bfe75cd 
This was missing some events before (reviews without comments, PR updated from a draft -> ready for review) so these were being ignored when finding the latest event. This PR adds them and restructures the code a bit to make it more clear what is happening for each PR. This addresses some of the issues from apache#9983
driazati added a commit to driazati/tvm that referenced this issue Feb 11, 2022
@driazati
[ci] Check more events before pinging reviewers
309127a 
This was missing some events before (reviews without comments, PR updated from a draft -> ready for review) so these were being ignored when finding the latest event. This PR adds them and restructures the code a bit to make it more clear what is happening for each PR. This addresses some of the issues from apache#9983
areusch pushed a commit that referenced this issue Feb 24, 2022
@driazati
[ci] Check more events before pinging reviewers (#10208)
35df464 
* [ci] Check more events before pinging reviewers

This was missing some events before (reviews without comments, PR updated from a draft -> ready for review) so these were being ignored when finding the latest event. This PR adds them and restructures the code a bit to make it more clear what is happening for each PR. This addresses some of the issues from #9983

* fix tests

Co-authored-by: driazati <driazati@users.noreply.github.com>
pfk-beta pushed a commit to pfk-beta/tvm that referenced this issue Apr 11, 2022
@driazati
[ci] Check more events before pinging reviewers (apache#10208)
ca99420 
* [ci] Check more events before pinging reviewers

This was missing some events before (reviews without comments, PR updated from a draft -> ready for review) so these were being ignored when finding the latest event. This PR adds them and restructures the code a bit to make it more clear what is happening for each PR. This addresses some of the issues from apache#9983

* fix tests

Co-authored-by: driazati <driazati@users.noreply.github.com>
@Mousius Mousius pinned this issue May 25, 2022
@Mousius Mousius unpinned this issue Jun 16, 2022
@driazati driazati mentioned this issue Sep 15, 2022
Merged
driazati added a commit to driazati/tvm that referenced this issue Sep 15, 2022
@driazati
[ci] Remove author check from ping bot
2eb0464 
This has been working fine for a while, this code opens it up so it's
not limited to the authors in apache#9983.
areusch pushed a commit that referenced this issue Sep 15, 2022
@driazati
[ci] Remove author check from ping bot (#12788)
5b43c62 
This has been working fine for a while, this code opens it up so it's
not limited to the authors in #9983.

Co-authored-by: driazati <driazati@users.noreply.github.com>
@areusch areusch added the needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it label Oct 19, 2022
@driazati driazati added dev:tools and removed needs-triage PRs or issues that need to be investigated by maintainers to find the right assignees to address it labels Oct 19, 2022
xinetzone pushed a commit to daobook/tvm that referenced this issue Nov 25, 2022
@driazati @xinetzone
[ci] Remove author check from ping bot (apache#12788)
6ea74f4 
This has been working fine for a while, this code opens it up so it's
not limited to the authors in apache#9983.

Co-authored-by: driazati <driazati@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
dev:tools type:rfc-tracking RFC progress tracking. Ref: https://github.com/apache/tvm-rfcs
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@areusch @driazati