Attention is currently required from: laforge.
1 comment:
Patchset:
I think this must be possible to solve more elegantly or at least in my mind more pythonic. […]
I agree that this is not elegant, and in the other comment i argue more towards dropping this patch entirely, but in case we don't drop it:
i think the simplest approach to serve the purpuse is to make one list containing all the "concrete" classes implemented in this file. like
ALL_CONFIGURABLE_PARAMETERS = [Imsi, Iccid, ....]
the fixed list because: it's questionable whether magically using all implemented subclasses is a good idea for anyone. I mean, *any* merged patch or imported module that happens to contain *any* weird experimental kind of ConfigurableParameter subclass suddenly appears in a magic discovery list. It's like a global variable. Do I want that to mess with my eSIM profiles in unexpected ways? No, so the list should be static, not magic.
i.e. anything using inheritance discovery is IMO not a good idea and it should be static.
To view, visit change 40095. To unsubscribe, or for help writing mail filters, visit settings.