Attention is currently required from: laforge.
fixeria has posted comments on this change. ( 
https://gerrit.osmocom.org/c/pysim/+/27130 )

Change subject: ts_102_221: Implement proper parsing of EF.DIR
......................................................................


Patch Set 6:

(1 comment)

File pySim/ts_102_221.py:

https://gerrit.osmocom.org/c/pysim/+/27130/comment/19633b3f_a7ed3f42
PS6, Line 512: ApplicationLabel
> the code actually works as expected, it is just pylint that complains about 
> it.
Good to know it works when passed to the metaclass (or whatever it's called).
Then adding a special comment should do the trick (untested):

  class ApplicationTemplate(BER_TLV_IE, tag=0x61, '# pylint: disable=E0602'
                            nested=[ ... ])



--
To view, visit https://gerrit.osmocom.org/c/pysim/+/27130
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings

Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I875eb49e1f0370428c2eae69af84f5483bd5b1fc
Gerrit-Change-Number: 27130
Gerrit-PatchSet: 6
Gerrit-Owner: laforge <[email protected]>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <[email protected]>
Gerrit-Attention: laforge <[email protected]>
Gerrit-Comment-Date: Fri, 11 Feb 2022 11:56:38 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No
Comment-In-Reply-To: laforge <[email protected]>
Comment-In-Reply-To: fixeria <[email protected]>
Gerrit-MessageType: comment

Reply via email to