[chirp_devel] Build test results: Successful

Jim Unroe
Sun Mar 15 15:04:27 PDT 2015


On Sun, Mar 15, 2015 at 5:47 PM, Dan Smith <dsmith at danplanet.com> wrote:
>> When it does the
>>
>> python tools\cpep8.py
>>
>> at the end, the computer pauses while I assume the script is doing
>> something, but it finishes without displaying anything. Is that
>> normal?
>
> Yep, that means you're good. If you go change something to break a rule
> and re-run it you should see an error. For example, if you make this change:
>
>> diff -r d0afd97ebb04 chirp/drivers/uv5r.py
>> --- a/chirp/drivers/uv5r.py     Sun Mar 15 13:16:48 2015 -0700
>> +++ b/chirp/drivers/uv5r.py     Sun Mar 15 14:46:17 2015 -0700
>> @@ -276,7 +276,7 @@
>>
>>  # 0x1EC0 - 0x2000
>>
>> -vhf_220_radio = "\x02"
>> +vhf_220_radio= "\x02"
>>
>>  BASETYPE_UV5R = ["BFS", "BFB", "N5R-2", "BTS"]
>>  BASETYPE_F11 = ["USA"]
>
> The pep8 checker will complain:
>
> ./chirp/drivers/uv5r.py:279:14: E225 missing whitespace around operator
>
> That means "Line 217, character 14, there should be a space before the =
> operator".
>
> --Dan

Outstanding. I think I will like that.

Jim



More information about the chirp_devel mailing list