Size: a a a

2021 September 02

RB

Roman Bolkhovitin in rannts
посмотри на flakehell, он умеет в легаси и ругаться только на измененные строчки
источник

AS

Artem Savinov in rannts
ой, отстань- нафиг мне туда смотреть- кроме этого оптимизатора- оно никому не надо
источник

AS

Artem Savinov in rannts
* но спасибо)
источник

SB

Sergey Belash in rannts
C++ программисты которые по полчаса ждут компиляции проекта вышли из чата 😁
источник

БС

Байт Словович... in rannts
ИМХО, роль линтеров в CI немного не правильно понимается.
Линтеры всё же должны помогать поддерживать качество кода, а не заниматься унижением разработчиков.
То бишь критерий фейла билда должен быть не наличие варнинга в файле / измененной строчке, а в выполнении следующих условий:
* отсутствие критических ошибок в любом месте файла (например, использование не инициализированной переменной)
* отсутствие легко исправимых варнингов, типа неисопльзуемый импорт, не используемый код и т.п.
* число варнингов с уровнем INFO (например Too broad exception clause  или Access to a protected member или method can be static) должно быть не больше 10 и не должно увеличиться больше, чем на 1 по сравнению с предыдущем билдом
* То же самое для минорных сообщений, но их может быть чуть больше как во всём файле, так и в изменениях

Ну плюс во время ревью ВСЕ линтеровские варнинги должны подсвечиваться. Пусть ревьютор принимает решение, что это ок или нет.

Если фейлить билд из за любого варнинга, то код будет просто тупо пестрить noinspection. У разработчиков разовьются комплексы и апатия и они по дефолту будут вставлять noinspection не думая.
источник

SB

Sergey Belash in rannts
Хороший доклад на тему, кстати. Несмотря на дикое название.
https://www.youtube.com/watch?v=b26Pni9N3NU&ab_channel=PythonChannel
источник

💭П

💭 Руслан Прохоров... in rannts
У него есть более ранние версии
источник

SA

Sergey Arkhipov in rannts
Ну и пускай. Во время код ревью всегда можно спросить, нафига и что ты глушишь. Плюс, особо шумные ошибки можно игнорировать в конфиге и тп
источник

SA

Sergey Arkhipov in rannts
Предлагаю стрелять в коленку за слово "вкусный" вне контекста еды
источник

БС

Байт Словович... in rannts
если игнорировать ошибки через конфиг, то можно пропустить реальную проблему.
источник

SB

Sergey Belash in rannts
Да, это печально. Но доклад от этого хуже не становится.
источник

БС

Байт Словович... in rannts
первые 10 минут доклада мне нравятся, я примерно так проповедую. не вижу ничего плохого в однобуквенных переменных в маааленьком контексте. Типа лямбы, forы и т.д
источник

RB

Roman Bolkhovitin in rannts
Что там? Про цикломатическую сложность рассказывает? ))
источник

БС

Байт Словович... in rannts
в контексте однобуквенных переменных упомянул. сделал так, что однобуквенная переменаая увеличивает сложность.
источник

A🌚

Al 🌚l in rannts
А ещё можно кодовую базу сжимать так) ведь кто-то всерьёз этот аргумент в споре табов и пробелов использует (спасибо кремниевой долине)
источник

БС

Байт Словович... in rannts
Хм, я уже лет 10 не видел споров на тему табов.
источник

A🌚

Al 🌚l in rannts
Всплывают то тут, то там)
источник

SZ

Sergey Z in rannts
Так говорят о другом и по весне :)
источник
2021 September 03

💭П

💭 Руслан Прохоров... in rannts
💭 Исполняющий обязанности бота информирует:

@Begemot2500,  у нас принято представляться и кратко рассказывать о себе с тэгом #whois.

Cпасибо за внимание!
источник

SB

Sergey Belash in rannts
Захотелось перенести эти аннотации в отдельный файл, с расширением .h (или .pyh)

https://adamj.eu/tech/2021/05/29/python-type-hints-how-to-use-overload/
источник