← Back to team overview

duplicity-team team mailing list archive

Re: [Merge] lp:~aaron-whitehouse/duplicity/08-adorn-strings into lp:duplicity

 

On 20.07.2018 12:46, Aaron wrote:
> Thanks Ede,
> 
> On 2018-07-20 10:21, edso wrote:
>> question. does it really make sense to adorn singular strings meant as
>> a code comment only eg.
>>
>>  class GlobbingError(Exception):
>> -    """Something has gone wrong when parsing a glob string"""
>> +    u"""Something has gone wrong when parsing a glob string"""
>>      pass
>>
>> ..ede
> 
> It's a fair question. I certainly do not see any harm, but potentially there is no benefit either -- are there any situations that the unclear encoding of a docstring could cause us problems?
> 
> If not and people would rather not add these, I can modify the test in find_unadorned_strings.py to exclude docstrings (i.e. strings starting with ' """ ' the line after a def).
> 
> Thoughts?

naa.. too much hassle. leave it in, does not really help but hurt either. was just curious.. ede/duply.net


References