----------------------------------------------------------- This is an automatically generated e-mail. To reply, visit: https://reviews.apache.org/r/57952/#review171180 -----------------------------------------------------------
src/cli_new/lib/mesos/plugins/config/main.py Lines 96-102 (patched) <https://reviews.apache.org/r/57952/#comment244060> I would actually leverage the Table abstraction to print th eplugins here. I think this might be a discarded RR from me that should probably be revived for this. - Kevin Klues On March 27, 2017, noon, Armand Grillet wrote: > > ----------------------------------------------------------- > This is an automatically generated e-mail. To reply, visit: > https://reviews.apache.org/r/57952/ > ----------------------------------------------------------- > > (Updated March 27, 2017, noon) > > > Review request for mesos and Kevin Klues. > > > Bugs: MESOS-7269 > https://issues.apache.org/jira/browse/MESOS-7269 > > > Repository: mesos > > > Description > ------- > > Used to load and show the plugins available for the user. > > > Diffs > ----- > > src/cli_new/bin/main.py bbfb52c894540158c70e0f50ebb8a277b692d54d > src/cli_new/bin/settings.py PRE-CREATION > src/cli_new/lib/mesos/plugins/config/__init__.py PRE-CREATION > src/cli_new/lib/mesos/plugins/config/main.py PRE-CREATION > > > Diff: https://reviews.apache.org/r/57952/diff/2/ > > > Testing > ------- > > Tested manually, PEP8 and Pylint used to make sure that the code style is > correct. > > > Thanks, > > Armand Grillet > >
