search

micropython / micropython-lib

Core Python libraries ported to MicroPython
Other
2.3k stars 980 forks source link

all: Enable ruff F841 'Local variable is assigned to but never used'. #856

Closed projectgus closed 1 month ago

projectgus commented 1 month ago

Most of these look like they were used for print debugging and then kept in when the print statements were removed or commented.

Some look like missing or incomplete functionality, these have been marked with comments where possible.

This PR shouldn't change runtime behaviour at all, however I can't rule out that one of these warnings wasn't masking an existing bug.

dpgeorge commented 1 month ago

Thank you! Looks good to me.