Attention is currently required from: dexter.

Patch set 1:Code-Review -2

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. […]

    It's actually recommended in PEP8 to separate top-level classes and methods using two empty lines, see https://peps.python.org/pep-0008/#blank-lines.

• File pySim/transport/__init__.py:

  • Patch Set #1, Line 28:

    I'm not quite sure what you are trying to achieve here. […]

    Acknowledged

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