Attention is currently required from: laforge.

Patch set 2:Code-Review +1

View Change

2 comments:

• File src/osmocom/utils.py:

  • Patch Set #2, Line 141: hexstr

    Class names should use CamelCase in python (according to PEP 8):

    https://peps.python.org/pep-0008/#class-names

    Built-in Python's classes not always comply to this, but I believe user defined ones should?

  • Patch Set #2, Line 151: other

    `... other: str) -> bool`?

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