You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/04/12 17:09:23 UTC

[GitHub] [tvm] areusch commented on pull request #7813: [µTVM] Add support for mps2_an521 board

areusch commented on pull request #7813:
URL: https://github.com/apache/tvm/pull/7813#issuecomment-817978760


   > @maheshambule @areusch Thanks a lot for the hint on how to check lint locally. I was not aware of them : ) `make format` actually suggested more than 200 changes, which really seemed too picky and not necessary, so I ended up using just `docker/bash.sh tlcpack/ci-lint:v0.62 ./tests/scripts/task_lint.s` and it made the magic.
   
   yeah so this is why you should prefer `docker/lint.sh` (or equivalent `docker/bash.sh`) for linting operations--the output of the lint tool is extremely sensitive to the version of e.g. `cpplint`, `pylint`, etc that you are using. the docker container removes that issue from consideration.


-- 
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.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org