[l2tdevtools] Changes for pylint 1.7 upgrade #163 and making Unicode strings the default #151

[l2tdevtools] Changes for pylint 1.7 upgrade #163 and making Unicode strings the default #151

[onager, 2017-08-04 10:04:36]

https://codereview.appspot.com/321340043/diff/1/utils/pylintrc
File utils/pylintrc (right):

https://codereview.appspot.com/321340043/diff/1/utils/pylintrc#newcode200
utils/pylintrc:200: argument-name-hint=(([a-z][a-z0-9_]*)|(_[a-z0-9_]*))$
The default looks better to me here - we shouldn't have very long or very short
argument names.

Also, this looks it can be simplified to just:
([a-z0-9_]*)

[onager, 2017-08-04 10:05:04]

On 2017/08/04 10:04:36, onager wrote:
> https://codereview.appspot.com/321340043/diff/1/utils/pylintrc
> File utils/pylintrc (right):
> 
> https://codereview.appspot.com/321340043/diff/1/utils/pylintrc#newcode200
> utils/pylintrc:200: argument-name-hint=(([a-z][a-z0-9_]*)|(_[a-z0-9_]*))$
> The default looks better to me here - we shouldn't have very long or very
short
> argument names.
> 
> Also, this looks it can be simplified to just:
> ([a-z0-9_]*)

LGTM to get pylint working again.

[Joachim Metz, 2017-08-05 05:50:51]

https://codereview.appspot.com/321340043/diff/1/utils/pylintrc
File utils/pylintrc (right):

https://codereview.appspot.com/321340043/diff/1/utils/pylintrc#newcode200
utils/pylintrc:200: argument-name-hint=(([a-z][a-z0-9_]*)|(_[a-z0-9_]*))$
On 2017/08/04 10:04:36, onager wrote:
> The default looks better to me here - we shouldn't have very long or very
short
> argument names.

Let's discuss this what the preferred lengths are and change that at later time.

> 
> Also, this looks it can be simplified to just:
> ([a-z0-9_]*)

ack

[Joachim Metz, 2017-08-05 06:28:54]

Changes have been merged with master branch. To close the review and clean up
the feature branch you can run: python ./utils/review.py close pylint