Да вроде ничего нехорошего нет. Отваливается - эксепшн внутри логики, обернуть ее в try.. except,  и выводить в лог если что то пошло не так.

> делаю вот такое, скорее всего нехорошее и неправильное
тебе надо что то сделать со своей низкой самооценкой...
1. Все ошибаются
2. Все говнокодят.
3. Прогеры из яндекса, гугла, амазона, говнокодят как и все.
4. Ты даже пришел на подкаст с Серегой и другими достаточно известными перцами.

PS Код нормальный, его можно увидеть в любом проекте, где есть async.  Раньше модно было делать через низкоуровнего апи аля asyncio.call_later и т.д. но славу богу, мэинтейнеры всех убедили, что оно нам не нужно.

да то был же ритуальный танец вроде "не бейте сильно тапками", и спасибо за ответы :)

Если через except Exception не ловится ошибка внутри "логики", попробуй завернуть в
except BaseException (может у тебя там где-то случается Canceled)
Но это только для отладки. В продакшен вот так не надо. Только если явно ловить Canceled, запускать какие-то логеры, деструкторы и дальше этот Canceled прокидывать

товарищи, мне один Тимлид ТИмлидов втирает что у них еще в далеком 2014 году flake8 натравливася только на измененные строки в МР.
По моему  он наркоман или все таки я?

* не представляю как линтер будет "оценивать" кусок из середины метода к примеру

Да ладно?

не знаю как flake но обычно линтеры запускаются на целый файл / модуль,
А потом выгребаются ошибки, только в измененных строках

во типа кинул

Блин. А я думал, что там всё просто. Костыли и мы можем делать )

ну там мысль то flake8 на diff натравить- вообще насколько эт оадекватно?

у ха ха

git diff -U0 | flake8 --diff

Это трындец. Это не отловит почти ничего

не испльзуемые переменные, ни импорты..

а какой смысл вообще на дифф его запускать?

вот к вам и пришел с таким вопросом, по моему тоже нет смысла

судя по доке, он все же прогоняет весь файл, но репортит варнинги только на изменные строки, которые получает из stdout

точно не путаешь?
там в прекомит хуке уже только diff

не вижу в описании чтобы ключик —diff для flake весь файл смотрел

хз что там в хуках..
https://flake8.pycqa.org/en/latest/user/invocation.html

  --diff                Report changes only within line number ranges in the
                        unified diff provided on standard in by the user.