Move pre-commit hooks to use Ruff #364
Conversation
✅ Deploy Preview for robyn canceled.
|
| @@ -5,14 +5,10 @@ repos: | |||
| hooks: | |||
| - id: black | |||
| args: [--line-length=160] | |||
There was a problem hiding this comment.
is 160 really what we want? :)
There was a problem hiding this comment.
ah, btw, the default_stages at the top might not be what we'd like, @sansyrox was there a particular reason it is only set to push?
There was a problem hiding this comment.
is 160 really what we want? :)
This was just something random. I just feel that 70 characters is a bit too less :p
ah, btw, the default_stages at the top might not be what we'd like, @sansyrox was there a particular reason it is only set to push?
@patrick91 , I am sorry but I don't understand what you mean by this. Can you please explain a bit more? 😅
There was a problem hiding this comment.
ah, btw, the default_stages at the top might not be what we'd like, @sansyrox was there a particular reason it is only set to push?
Now, I understand. It was mistake 🤣
|
Thank you @patrick91 😄 . I have a few questions on the PR. |
Description
Ruff is a much faster linter (written in rust), it also supports
sorting imports (replacing isort) and fixing issues.