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.

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

[ci] Move auto-cc bot edits on PRs to a comment #12637

Closed
2 of 7 tasks
driazati opened this issue Aug 29, 2022 · 0 comments
Closed
2 of 7 tasks

[ci] Move auto-cc bot edits on PRs to a comment #12637

driazati opened this issue Aug 29, 2022 · 0 comments
Assignees
Labels
dev:ci type:ci Relates to TVM CI infrastructure

Comments

@driazati
Copy link
Member

driazati commented Aug 29, 2022

To enable the use of https://github.blog/changelog/2022-08-23-new-options-for-controlling-the-default-commit-message-when-merging-a-pull-request/ so we can have good defaults for commit messages, we need to ensure that PR descriptions don't contain @ tags of GitHub users. We can lint for this but the auto-cc bot edits PR descriptions to @ people on PRs, so we need to change this bot to leave a comment instead. Right now we have other bots that leave comments on PRs, this would be a good time to refactor them a bit and merge everything into 1 big bot comment. This could also contain helpful links for contributors and let us get rid of the pr template.

To implement the lint check there are a few sequential PRs:

Then to implement the unified bot comment on PRs:

cc @Mousius @areusch @gigiblender @gromero

@driazati driazati added the type:ci Relates to TVM CI infrastructure label Aug 29, 2022
@driazati driazati self-assigned this Aug 29, 2022
@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
@janetsc janetsc added dev:ci 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
@driazati driazati closed this as completed Dec 6, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dev:ci type:ci Relates to TVM CI infrastructure
Projects
None yet
Development

No branches or pull requests

3 participants