fish-shell/share/tools
Kurtis Rader 7bb3bf7c74 fix regression from commit 20bcbcc2
There were two places in the code that used the anti-pattern of
returning True on success else an error message. In python you should
always be able to replace `if x == True:` with just `if x:`. Which is
what the lint tool recommended. Unfortunately I didn't notice how the
return value was being used. This fixes that by changing the two
affected functions to return an error message or None on success.

This also adds `from __future__ import print_function` since the code
uses the `print(msg)` function form rather than the `print msg`
statement form. The former works by accident on python2 because the
parens are interpreted as creating parenthesized expression that
devolves to the single string inside the parens. So while the future
import isn't strictly speaking necessary it will help avoid mistakes in
the future if more complex `print()` calls are added.

Partial fix for #3620
2016-12-04 16:01:32 -08:00
..
web_config fix regression from commit 20bcbcc2 2016-12-04 16:01:32 -08:00
create_manpage_completions.py simplify invoking create_manpage_completions.py 2016-10-09 18:42:55 -07:00
deroff.py simplify invoking create_manpage_completions.py 2016-10-09 18:42:55 -07:00