laforge submitted this change.

View Change

Approvals: laforge: Looks good to me, approved fixeria: Looks good to me, but someone else must approve Jenkins Builder: Verified
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(-)

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 change 39748. To unsubscribe, or for help writing mail filters, visit settings.

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