On Tue, Jan 15, 2013 at 4:24 PM, J <[email protected]> wrote:
> The exit code determination above works, but it just feels inelegant.
> It feels like there's a better way of implementing that, but I can't
> come up with one that still honors the fail level properly (e.g. other
> solutions will fail on medium, but won't fail properly on medium OR
> higher).
First, instead of having separate variables 'critical_fails',
'high_fails', etc., put them in a collections.Counter 'fails' keyed by
fail level.
Second, make sure those fail level keys are orderable by severity.
Then your check is just:
if fail_priority <= max(fails):
return 1
return 0
--
http://mail.python.org/mailman/listinfo/python-list
