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 <[email protected]>

Reply via email to