We discussed this at the TVM Community Meeting this morning.
- It seems like some of the problems addressed here (unable to assign PRs to
folks outside the Apache org) are better handled by adding more folks to Apache
org. Are we exploring this?
- Yes, but separately from this automation. T
Posted the https://github.com/apache/tvm-rfcs/pull/58 with some links to PRs
that should address some of the concerns.
---
[Visit Topic](https://discuss.tvm.apache.org/t/rfc-remove-codeowners/12095/13)
to respond.
You are receiving this because you enabled mailing list mode.
To unsubscri
Thanks for @driazati @areusch to bring this for discussion, I think it make
sense to reduce the notification spam by rename the codeowner file, and
definitely that can help contributor to focus on more related message from
community.
I agree with @comaniac @manupa-arm about helping the new
I see. All good then :D
---
[Visit Topic](https://discuss.tvm.apache.org/t/rfc-remove-codeowners/12095/11)
to respond.
You are receiving this because you enabled mailing list mode.
To unsubscribe from these emails, [click
here](https://discuss.tvm.apache.org/email/unsubscribe/e723fbd60a6
I think that's what @driazati meant. "Removing" CODEOWNERS doesn't mean to
remove but rename that file: https://github.com/apache/tvm/pull/10192
---
[Visit Topic](https://discuss.tvm.apache.org/t/rfc-remove-codeowners/12095/10)
to respond.
You are receiving this because you enabled mailin
@driazati @areusch ,
This looks like a great suggestion!.
I think the proposal is about adding a mechanism to use the cc tag to attach
people as reviewers, which seems good step.
I agree with @comaniac by helping the new authors finding people to tag for
reviews rather than doing a mandator
[quote="areusch, post:7, topic:12095"]
I agree we have this, I’m not sure it scans very well (for example, the links
are in Markdown so someone isn’t going to mentally parse those and render them
in their head). Perhaps we can improve the template and make it a bit more
organized.
[/quote]
@comaniac thanks for your comments!
[quote="comaniac, post:5, topic:12095"]
Improve the PR template to guide the contributors about how to improve the PR
visibility. We actually have this already but seems not effective. Pro: it is a
simple change. Con: it may be ignored by most contributors.
I agree that the existing approach does not work well and support this RFC :+1:
---
[Visit Topic](https://discuss.tvm.apache.org/t/rfc-remove-codeowners/12095/6)
to respond.
You are receiving this because you enabled mailing list mode.
To unsubscribe from these emails, [click
here](https
Supportive. It's tedious that Apache org creates many barriers for us to
leverage a more systematic way, but the proposed solutions with a number of
bots to assist the review process are definitely helpful.
Meanwhile, I recall that one important reason of adding code owners is to help
improv
It definitely makes sense for us to reduce the traffic from the github emails.
Github teams is definitely a good idea. I'm supportive 👍
---
[Visit Topic](https://discuss.tvm.apache.org/t/rfc-remove-codeowners/12095/4)
to respond.
You are receiving this because you enabled mailing list mod
I support this RFC and will shepherd it through review process. It looks like
we have some :heart: from the OctoML folks, but would be great to get some
feedback from others in the community, particularly from folks who may be
monitoring TVM PRs less frequently than those of us at Octo.
Let's
It was brought up that it would be nice to have the ability to not only cc
specific people as reviewers but groups of people by labels. PyTorch does
something [similar](https://github.com/pytorch/pytorch/issues/24422) which we
could mostly copy. So labels could be automatically assigned based
# **Summary**
Move `.github/CODEOWNERS` to `.github/CODEOWNERSHIP` to avoid triggering
GitHub’s [automatic review
requests](https://docs.github.com/en/repositories/managing-your-repositorys-settings-and-features/customizing-your-repository/about-code-owners#about-code-owners).
Add GitHub Acti
14 matches
Mail list logo
