Attention is currently required from: fixeria, neels.

laforge has posted comments on this change by neels. ( 
https://gerrit.osmocom.org/c/pysim/+/39741?usp=email )

Change subject: [1/6] personalization: refactor: drop ClassVarMeta use
......................................................................


Patch Set 6:

(1 comment)

Patchset:

PS6:
I'm not entirely sure why the new approach is considered better or an advantage 
over the old one.  For example, the 'name' attribute is no longer being set 
automatically, which is something I wanted to use for CSV based 
personalization.  Keep in mind this code was not written for the eSIM 
personalization web platform but with other use cases in mind.



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

Gerrit-MessageType: comment
Gerrit-Project: pysim
Gerrit-Branch: master
Gerrit-Change-Id: I60ea8fd11fb438ec90ddb08b17b658cbb789c051
Gerrit-Change-Number: 39741
Gerrit-PatchSet: 6
Gerrit-Owner: neels <nhofm...@sysmocom.de>
Gerrit-Reviewer: Jenkins Builder
Gerrit-CC: fixeria <vyanits...@sysmocom.de>
Gerrit-CC: laforge <lafo...@osmocom.org>
Gerrit-Attention: neels <nhofm...@sysmocom.de>
Gerrit-Attention: fixeria <vyanits...@sysmocom.de>
Gerrit-Comment-Date: Mon, 10 Mar 2025 17:00:12 +0000
Gerrit-HasComments: Yes
Gerrit-Has-Labels: No

Reply via email to