ненависть

[amarao_san]

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

Comments

[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]

Попробуй вот так:

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 до оргазма.

flake8 neutron/tests/unit/openvswitch/test_ovs_lib.py
neutron/tests/unit/openvswitch/test_ovs_lib.py:490:17: E126 continuation line over-indented for hanging indent
["ovs-vsctl", self.TO, "--format=json", "--",
^
neutron/tests/unit/openvswitch/test_ovs_lib.py:493:13: E121 continuation line under-indented for hanging indent
).AndReturn(self._encode_ovs_json(headings, data))
^

[eterevsky]

Ок, признаю, я не понимаю, чего он хочет. Его что ли смущает что у разных частей строчки разные инденты?

[amarao_san]

Он хочет, чтобы текст находился правильно. В случае слишком глубокой вложенности его начинает глючить. Приходится хачить.