lhutton1 opened a new issue, #11785: URL: https://github.com/apache/tvm/issues/11785
Recently the version of PyLint was updated from `2.4.4` to `2.9.3` ([#11733](https://github.com/apache/tvm/issues/11733)), however, in doing so usage of the new `ci_lint` image are failing with a bunch of linting complaints e.g. https://ci.tlcpack.ai/job/docker-images-ci/job/daily-docker-image-validate/146/. While this is not a problem in TVM's CI since the images need updating manually (for now), it renders using a newly built `ci_lint` unusable due to the linting complaints. One solution would be to go through and fix all of these complaints, although since there are quite a few (~500) it may take some time to get the codebase upto the standard of the new PyLint version. Therefore, I suggest we revert the commit #11733 for now so that a newly build `ci_lint` image can be used by anyone wishing to do so. Then, submit a new patch with the version change and the relevant complains fixed at a later date. Wondering what other people's thoughts are on this? cc @leandron @manupa-arm @areusch @AndrewZhaoLuo @junrushao1994 @ashutosh-arm -- 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]
