ненависть

[amarao_san]

Я понимаю, когда нужно довести до оргазма pep8/flake8. Но проблема состоит в том, что у меня на компьютере и на jenkins'е разные pep'ы (и да, я пробовал их ставить через easy_intall - всё равно не та версия). И они возбуждаются на разное. А я не понимаю, как удовлетоврить удалённый pep, потому что у него не встаёт на то, что встаёт у локального pep

Comments

[kostoprav]

наплодить пару лишних переменых нэ?
типа
cmd = ["ovs-vsctl", self.TO, ..., "Interface"]
payload = self._encode_ovs_json(headings, data)

utils.execute(cmd, root_helper=self.root_helper).AndReturn(payload)

Хотя я вобще не любитель function chaining

[amarao_san]

Это не совсем мой кот, а бэкпорт, так что я его стремился сохранить максимально близко к оригиналу.

[kostoprav]

если парой движений, не внося новых зависимостей или сайд эффектов можно улучшить мир, почему нет?
Ну и плюс, дифф по строчкам примерно такой-же будет, а следующий человек потратит меньше времени на чтение.

[amarao_san]

Потому что это бэкпорт. Он не пойдёт в апстрим. В апстриме он уже есть. Его жизнь - это до конца срока поддержки продукта. В следующей версии этого кода уже не будет.

А вот следующий бэкпортящий может долго тупить на том, что "тут переменная есть, а тут нет". Так что чем меньше бэкпорт меняет, тем лучше.

При этом нужно найти балланс между "меняет в старом коде" и "меняет что-то в перенесённом новом коде".

[eterevsky]

Я бы вот так форматировал:

utils.execute(
["ovs-vsctl", self.TO, "--format=json", "--",
"--columns=name,external_ids", "list", "Interface"],
root_helper=self.root_helper
).AndReturn(self._encode_ovs_json(headings, data))

[amarao_san]

flake8 neutron/tests/unit/openvswitch/test_ovs_lib.py

neutron/tests/unit/openvswitch/test_ovs_lib.py:493:11: E121 continuation line under-indented for hanging indent
).AndReturn(self._encode_ovs_json(headings, data))

Там фашисты. Не прорвёшься.

[eterevsky]

Тогда поднять ).AndReturn(...) на indent 4, а параметры execute на indent 8. Меня помимо lint'а беспокоит, чтобы код был максимально читаем, а для этого иерархия должна максимально сохраняться.

И да, у нас тоже все коммиты на всех языка проверяются дико въедливым линтом.

[amarao_san]

Эм... не понял, что сделать.

Но, в любом случае, эта дрянь требует изуродовать код для прохода его style guide'а.

[eterevsky]

Кстати, PEP8 достаточно размыт в смысле описания стиля. Вот этот style guide более подробен и конкретен.

[amarao_san]

Ну, меня стиль мало волнует, а вот flake8 фейлит билды только так.

[eterevsky]

В тему: