search

nornir-automation / nornir

Pluggable multi-threaded framework with inventory management to help operate collections of devices
https://nornir.readthedocs.io/
Apache License 2.0
1.39k stars 235 forks source link

Remove unnecessary assignment before `return` statement #958

Closed ogenstad closed 3 months ago

ktbyers commented 3 months ago

Ruff is convincing me not to use it in other projects (LOL).

ogenstad commented 3 months ago

Ruff is convincing me not to use it in other projects (LOL).

Well it mostly reimplements rules from other linters, though it has a few rules of its own too. I think most of the rules make sense and adds clarity. As with other linkers it's always easier if it's in place when you start coding instead of going back and adding it later.

ktbyers commented 3 months ago

Yeah, it doesn't really matter (as with almost all linter things--it just matters that it is done/decided automatically).