flake8 >= 3.8.0 emits E741 ambiguous variable name 'l'
Affects | Status | Importance | Assigned to | Milestone | |
---|---|---|---|---|---|
curtin |
Fix Committed
|
High
|
Lucas Albuquerque Medeiros de Moura |
Bug Description
I couldn't find a relevant entry in the flake8 release notes [1], but flake8 3.7.9 does not emit the error, while 3.8.0 does. The fact that they were not being emitted was likely a flake8 bug, as the recommendation [2,3] causing the error is nothing new. The errors are:
./curtin/
./curtin/
./curtin/
./curtin/
./tests/
./tests/
./tests/
./tests/
I think renaming l to L would be enough, and won't require rewraps.
[1] https:/
[2] https:/
[3] https:/
Changed in curtin: | |
importance: | Undecided → High |
status: | New → Confirmed |
tags: | added: bitesize |
Changed in curtin: | |
assignee: | nobody → Lucas Albuquerque Medeiros de Moura (lamoura) |
Changed in curtin: | |
status: | Confirmed → Fix Committed |
Lucas' branch:
https:/ /code.launchpad .net/~lamoura/ curtin/ +git/curtin/ +merge/ 383811