secure-systems-lab / securesystemslib

Cryptographic and general-purpose routines for Secure Systems Lab projects at NYU
MIT License
46 stars 49 forks source link

linter: ruff coverage #834

Open jku opened 4 days ago

jku commented 4 days ago

The various linters have been replaced by ruff (thanks to @L77H).

As a initial list, these contain useful rules

I have not tested how much work enabling them would be -- starting with the easiest may be a good idea

L77H commented 4 days ago

I tested the initial list you proposed:

I believe this is a good starting point to be added, I will create a separate PR for both UP and ANN if you agree.

jku commented 4 days ago

Using PL as a rule includes the following: PLR, PLE, PLC, PLW so those are already covered

oh that makes sense -- I got tricked since pycodestyle is documented the same way but that one doesn't seem to have a "top-level" ruleset

Which reminds me: pycodestyle warnings (W) might be useful to enable as well

L77H commented 4 days ago

Yes the markup is a bit confusing. Good, I will add W to the TODO list.