Attention is currently required from: laforge.

View Change

1 comment:

• Patchset:

  • Patch Set #1:

    Likewise, I find this weird. I am also getting this with pylint v3.2.5.

    This makes the linter happy:

    ```diff
    diff --git a/pySim-shell.py b/pySim-shell.py
    index 174df62..6ee127e 100755
    --- a/pySim-shell.py
    +++ b/pySim-shell.py
    @@ -839,7 +839,8 @@ class PySimCommands(CommandSet):
             to get write/update permissions to most of the files on SIM cards.
             """
             if opts.adm_type:
    -            adm_chv_num = pin_names.inverse[opts.adm_type]
    +            pin_values = dict(pin_names.inverse)
    +            adm_chv_num = pin_values[opts.adm_type]
             else:
                 adm_chv_num = self._cmd.card._adm_chv_num
             if opts.ADM:
    ```

    But I think we can just do `pylint: disable=unsubscriptable-object`.

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