view setup.cfg @ 6306:075f365a53cf

topic: remove validator safeguard that wasn't safeguarding anything

The else block was simply setting origvalidator to None, but it's not used
unless hg is version <= 5.3, but in that case origvalidator is set to
tr._validator. So the block wasn't necessary, but also we can't call None as a
function, so it didn't make a lot of sense either way.

Also adding return to return the original validator return value. Mostly for
consistence, because validators either raise an exception or they don't, return
value currently doesn't seem to be used.

[flake8]
ignore =
         #closing bracket does not match indentation of opening bracket's line
         E123,
         # closing bracket does not match visual indentation
         E124,
         # visually indented line with same indent as next logical line
         E129,
         # at least two spaces before inline comment
         E261,
         # too many leading '#' for block comment
         E266,
         # expected 2 blank lines, found 0
         E302,
         # expected 2 blank lines after end of function or class
         E305,
         # module level import not at top of file
         E402,
         # line too long (82 > 79 characters)
         E501,
         # do not assign a lambda expression, use a def
         E731,
         # class names should use CapWords convention
         N801,
         # line break occurred before a binary operator
         W503
builtins=xrange, execfile