Attention is currently required from: dexter.

Patch set 1:Code-Review +1

View Change

2 comments:

• File pySim-shell.py:

  • Patch Set #1, Line 1131:

    I think it's useful to sepaate the "main" block from class definitions with an extra line. Not sure why you think this needs our attention or "fixing".

• File pySim/transport/__init__.py:

  • Patch Set #1, Line 28:

    I'm not quite sure what you are trying to achieve here. I think it's not uncommon to have an empty line to separate imporst from foreign modules to imports from within a project (pysim here).

To view, visit change 42733. To unsubscribe, or for help writing mail filters, visit settings.