Mousius commented on PR #13916: URL: https://github.com/apache/tvm/pull/13916#issuecomment-1420894409
> The idea is that even though things may not have been caught immediately by one of us(such as test coverage), having a quick followup comment such as (please add test coverage) will likely result in a quick PR from the merger to update the test case, as a result we quickly get to the state of readiness. In this case, likely I would also suggested the same, and as a result covers the test. So such iterative engagement and feedbacks helps us to continue improve the changes. The door is not closed at merging point. > > There are also more ways that we can help to ramp up in the related code areas and bringing suggestions, participating in [community forums](https://discuss.tvm.apache.org/c/development/unity/14), bringing in suggestions are all valuable ways to help us continue to improve the set of changes over the branch. Thanks for the clarification @tqchen, it is clear that the `unity` branch is not intended to be as inclusive of the wider community as the `main` branch and favours specific kinds of contributors; this PR and https://github.com/apache/tvm/pull/13926 are clear indicators of that, and it is deeply saddening that we have existing guidelines to help make TVM more inclusive yet the `unity` branch has chosen not to follow them for such a large contribution. -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: commits-unsubscr...@tvm.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
