quic-sanirudh commented on PR #11712: URL: https://github.com/apache/tvm/pull/11712#issuecomment-1189822698
Is there any ideas to either merge this patch, or modify this to remote all the single variable names altogether. When modifying the pytests to be linter compliant, I normally just run the linter and fix the errors it reports (I guess others do it in a similar fashion). As it stands now, the linter errors for variable `z` but not for `x` and `y`, so we might end up missing a few bad variable names. Right now, I can manually change the linter regex in my local machine to avoid missing out on the bad variable names, but it might be good if we decided to go one way or another. -- 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]
