Fix Flake8's E741 errors (ambiguous variable name). This prevents
confusion when reading code with a font that does not distinguish
clearly between l, 1 and I.
Details
Diff Detail
- Build Status
Buildable 85371 Build 116552: arc lint + arc unit
Event Timeline
lnt/server/reporting/report.py | ||
---|---|---|
12 | lol, think it would have been about as much effort to fix :) |
lnt/server/reporting/report.py | ||
---|---|---|
12 | That's what I went for initially but change it for what? list is a keyword and creates more confusion since it's gonna be color-coded in any sensible IDE. And since the function is for any kind of list I'm not sure what else to use. I thought about "elems" but l is quite common for list comprehension AFAIK. I welcome suggestion, I'd prefer avoiding noqa hints. |
lnt/server/reporting/report.py | ||
---|---|---|
12 | how about src? I mentioned because like on Phab, the highlight is quite subtle. |
lol, think it would have been about as much effort to fix :)