laforge has uploaded this change for review. ( https://gerrit.osmocom.org/c/pysim/+/41736?usp=email ) Change subject: pySim.esim.saip.ProfileElementSequence: Update type annotations ...................................................................... pySim.esim.saip.ProfileElementSequence: Update type annotations The type annotations didn't reflect reality in two cases. Change-Id: Ib99c00a38bf009c63180b4a593d6cc796ff282d3 --- M pySim/esim/saip/__init__.py 1 file changed, 2 insertions(+), 2 deletions(-) git pull ssh://gerrit.osmocom.org:29418/pysim refs/changes/36/41736/1 diff --git a/pySim/esim/saip/__init__.py b/pySim/esim/saip/__init__.py index 6e8823b..f661745 100644 --- a/pySim/esim/saip/__init__.py +++ b/pySim/esim/saip/__init__.py @@ -1786,7 +1786,7 @@ return None @staticmethod - def peclass_for_path(path: Path) -> Optional[ProfileElement]: + def peclass_for_path(path: Path) -> (Optional[ProfileElement], Optional[templates.FileTemplate]): """Return the ProfileElement class that can contain a file with given path.""" naa = ProfileElementSequence.naa_for_path(path) if naa: @@ -1819,7 +1819,7 @@ return ProfileElementTelecom, ft return ProfileElementGFM, None - def pe_for_path(self, path: Path) -> Optional[ProfileElement]: + def pe_for_path(self, path: Path) -> (Optional[ProfileElement], Optional[templates.FileTemplate]): """Return the ProfileElement instance that can contain a file with matching path. This will either be an existing PE within the sequence, or it will be a newly-allocated PE that is inserted into the sequence.""" -- To view, visit https://gerrit.osmocom.org/c/pysim/+/41736?usp=email To unsubscribe, or for help writing mail filters, visit https://gerrit.osmocom.org/settings?usp=email Gerrit-MessageType: newchange Gerrit-Project: pysim Gerrit-Branch: master Gerrit-Change-Id: Ib99c00a38bf009c63180b4a593d6cc796ff282d3 Gerrit-Change-Number: 41736 Gerrit-PatchSet: 1 Gerrit-Owner: laforge <laforge@osmocom.org>