jan/masque
1
1
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Add NoReturn __contains__ with a more descriptive error message

Browse Source
This commit is contained in:
jan 2023-10-12 01:32:33 -07:00
parent c02c2f90ef
commit 3245de99b3
1 changed files with 4 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

5
masque/ports.py
View File

@ -1,4 +1,4 @@
from typing import Iterable, KeysView, ValuesView, overload, Self, Mapping from typing import Iterable, KeysView, ValuesView, overload, Self, Mapping, NoReturn
import warnings import warnings
import traceback import traceback
import logging import logging
@ -138,6 +138,9 @@ class PortList(metaclass=ABCMeta):
else: else:
return {k: self.ports[k] for k in key} return {k: self.ports[k] for k in key}
def __contains__(self, key: str) -> NoReturn:
raise NotImplementedError('PortsList.__contains__ is left unimplemented. Use `key in container.ports` instead.')
# NOTE: Didn't add keys(), items(), values(), __contains__(), etc. # NOTE: Didn't add keys(), items(), values(), __contains__(), etc.
# because it's weird on stuff like Pattern that contains other lists # because it's weird on stuff like Pattern that contains other lists
# and because you can just grab .ports and use that instead # and because you can just grab .ports and use that instead