AS
вот так
return cast(List[NetboxServiceResponse], get_data_from_netbox(url, params)) никто не ругаетсяSize: a a a
2021 July 31
SV
это чит
SV
проще просто не запускать mypy
AS
я как раз убирал всяческое дублирование и делал так что бы функция была одна. ибо она один в один
SV
можно вынести общий код в какую-нибудь непубличную функцию
AS
это и есть не публичная функция с общим кодом
SV
видимо нужно больше непубличных функций
AS
я не понимаю куда их больше
DA
А от чего зависит возвращаемый тип? От params?
Пробовал добавить assert на тип перед возвратом?
Пробовал добавить assert на тип перед возвратом?
AS
от url
AS
там за выкидываением лишнего
def get_services() -> List[NetboxServiceResponse]:
url = f"{settings.url}/api/ipam/services/"
params = ...
return cast(List[NetboxServiceResponse], get_data_from_netbox(url, params))
AS
функция которая про девайсы ну почти та же. но другой урл и параметры
DA
Наверное можно запариться с описанием типа и оверлодом, с проверкой по контенту url (хз правильно ли это) как здесь в примере с
https://mypy.readthedocs.io/en/stable/more_types.html
А может оставить и каст, но имхо перед этим стоит ассерт добавить типа:
result = get_data_from_netbox(...)
assert all(isinstance(Netbox..., result))
mouse_event сделано по типу https://mypy.readthedocs.io/en/stable/more_types.html
А может оставить и каст, но имхо перед этим стоит ассерт добавить типа:
result = get_data_from_netbox(...)
assert all(isinstance(Netbox..., result))
AS
я честно не уверен что это надо внутри
NetboxResult.parse_raw(r.content) построенный pydanticAS
если оно сломается то вероятно это отобъет пидантик
p
Я бы просто забил на типы и компенсировал тестами
AS
ну у меня код который вообще можно на всё забить. я скорее пытаюсь понять как по красоте
p
Я смотрю на все эти страдания и не хочу себе такого
DA
красота требует жертв)
AS
ну я согласен на жертвы то, но не согласен на карго культ