Attention is currently required from: laforge.
4 comments:
Patchset:
none of the comments from April adressed even in January of the following year.
I apologize if they have they been marked as resolved in error.
I assume you are stating these facts in order to make me feel supported and to assist in resolving a long standing blockage that has manifested not entirely without your involvement, and which i do not particularly enjoy, at all, for many many months now.
Or do you dislike when a branch is submitted to probe build status?
File pySim/esim/saip/param_source.py:
Patch Set #3, Line 33: abstract
I think the code has moved but the question remains. […]
plug here the same conclusion i formed about the other is_abstract flag.
I think we should have one static list, or no list at all in order to make callers have their own static list.
Patch Set #3, Line 132: RandomDigitSource
IMHO it's a bit unintuitive that the IncDigit inherits from RandomDigit. […]
ack
Patch Set #3, Line 133: 'incrementing sequence of digits'
cosmetic: the docstring here uses single-quotes, below for the method it uses double-quotes. […]
(i never got over it, python used to be so single-quote-y, but at some point linters started making me use double quotes, now my usage ends up completely chaotic, a mix of remembering and forgetting. i mean, you need SHIFT for double quotes...!)
To view, visit change 40096. To unsubscribe, or for help writing mail filters, visit settings.