laforge has submitted this change. ( 
https://gerrit.osmocom.org/c/pysim/+/39748?usp=email )

Change subject: pylint: personalization.py: fix E1135: permitted_len 
unsupported-membership-test
......................................................................

pylint: personalization.py: fix E1135: permitted_len unsupported-membership-test

pre-empt this from coming up in patch
I60ea8fd11fb438ec90ddb08b17b658cbb789c051:

E1135: Value 'self.permitted_len' doesn't support membership test 
(unsupported-membership-test) pickermitted_len

Change-Id: I0343f8dbbffefb4237a1cb4dd40b576f16111073
---
M pySim/esim/saip/personalization.py
1 file changed, 1 insertion(+), 1 deletion(-)

Approvals:
  laforge: Looks good to me, approved
  fixeria: Looks good to me, but someone else must approve
  Jenkins Builder: Verified




diff --git a/pySim/esim/saip/personalization.py 
b/pySim/esim/saip/personalization.py
index 61973e7..cd23944 100644
--- a/pySim/esim/saip/personalization.py
+++ b/pySim/esim/saip/personalization.py
@@ -112,7 +112,7 @@
     key_id = None
     kvn = None
     key_usage_qual = None
-    permitted_len = None
+    permitted_len = []

     def validate(self):
         if not isinstance(self.input_value, (io.BytesIO, bytes, bytearray)):

--
To view, visit https://gerrit.osmocom.org/c/pysim/+/39748?usp=email
To unsubscribe, or for help writing mail filters, visit 
https://gerrit.osmocom.org/settings?usp=email

Gerrit-MessageType: merged
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I0343f8dbbffefb4237a1cb4dd40b576f16111073
Gerrit-Change-Number: 39748
Gerrit-PatchSet: 3
Gerrit-Owner: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-Reviewer: fixeria <vyanits...@sysmocom.de>
Gerrit-Reviewer: laforge <lafo...@osmocom.org>

Reply via email to