tqchen commented on PR #13916: URL: https://github.com/apache/tvm/pull/13916#issuecomment-1420654749
Thanks @Mousius, thank you for your inputs . Agree that it is a challenge to be able to get ramped up and look into the set of changes just in a PR. The good part of branch development is that we have a long extended period of time, where multiple evidences, feedbacks, engagements can be provided, to get the community ramped up, participate, and review the set of changes as a whole through a long extended period of time. 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. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
