Attention is currently required from: laforge.

fixeria has posted comments on this change. ( https://gerrit.osmocom.org/c/pysim/+/37685?usp=email )

Change subject: pySim-shell: Support other ADMx values beyond ADM1 from 'verify_adm' ......................................................................

Patch Set 1:

(1 comment)

Patchset:

PS1:

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`.