Quoting Brian Paul (2017-10-16 12:49:43) > --- > framework/summary/console_.py | 7 ++++--- > 1 file changed, 4 insertions(+), 3 deletions(-) > > diff --git a/framework/summary/console_.py b/framework/summary/console_.py > index 3b58ba4..ee6cbd8 100644 > --- a/framework/summary/console_.py > +++ b/framework/summary/console_.py > @@ -105,10 +105,11 @@ def _print_result(results, list_): > statuses=' '.join(str(r) for r in results.get_result(test)))) > > > -def console(results, mode): > - """ Write summary information to the console """ > +def console(resultsFiles, mode):
Could we use pep8 style results_files instead of resultsFiles? With that changed for the series: Reviewed-by: Dylan Baker <dy...@pnwbakers.com> > + """ Write summary information to the console for the given list of > + results files in the given mode.""" > assert mode in ['summary', 'diff', 'incomplete', 'all'], mode > - results = Results([backends.load(r) for r in results]) > + results = Results([backends.load(r) for r in resultsFiles]) > > # Print the name of the test and the status from each test run > if mode == 'all': > -- > 1.9.1 > > _______________________________________________ > Piglit mailing list > Piglit@lists.freedesktop.org > https://lists.freedesktop.org/mailman/listinfo/piglit
signature.asc
Description: signature
_______________________________________________ Piglit mailing list Piglit@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/piglit
