On Fri, Apr 5, 2024 at 2:36 PM Dan Smith via Developers developers@lists.chirpmyradio.com wrote:
How much of a problem is this really? Any suggestions to make it pass tests?
It's a big problem, indentation is important in python as is reasonably consistent style in a project. The rules being enforced aren't even ours, they're from PEP8, which is the bare minimum standard formatting recommendation from the python project itself. The newer (and much uglier) set is about a billion times more strict (also about the same amount uglier too).
Just break the line at the open-paren, either for RadioSetting or RadioSettingValueBoolean, as is recommended by PEP8:
Thanks! That worked. Hadn't read PEP8 in a long time, so I forgot about the peculiar line length. Usually for bash or C-like languages I assume 80 chars unless specifically told otherwise.
Reviewing PEP8 gave me a lot to think about.
---- Charles Terrell tachyon@pobox.com