[blivet:master 04/20] Disable some W0703 warnings
Vratislav Podzimek
vpodzime at redhat.com
Tue Apr 15 07:29:24 UTC 2014
On Fri, 2014-04-11 at 14:03 -0400, mulhern wrote:
> These all warn about catching exception Exception in the context of an
> idiom where the failure is always logged and, sometimes, an optional default
> value is set.
I believe that pylint's warnings here have some sense in them. I've seen
a lot of cases where were catching things like "NoneType has no
attribute something" and just silently logging them or showing them to
users as "partitioning errors". These are obviously serious flaws in our
(or external) code. So we should really remove those "except Exception:"
statements and replace them with something better.
--
Vratislav Podzimek
Anaconda Rider | Red Hat, Inc. | Brno - Czech Republic
More information about the anaconda-patches
mailing list