tenstorrent / tt-metal

:metal: TT-NN operator library, and TT-Metalium low level kernel programming model.
Apache License 2.0
399 stars 50 forks source link

infra: implement linters for pre-commit #188

Open tt-rkim opened 1 year ago

tt-rkim commented 1 year ago

We want linters that

For the kinds of linters we may want now,

For the kinds of linters that we may want but need to do a little more investigating,

For Python specifically, I believe we should hold off on static type checkers. They're a bit too much overhead right now, and it may be difficult to setup. Ex. Not all libraries have type stubs that are useful/correct

tt-rkim commented 1 year ago

I would appreciate any comments from you guys @abhullar-tt @DrJessop @coffeeandcheesecake @tt-nshanker

tt-rkim commented 1 year ago

Still need this. Leaving open