CodeStyle

[GomZik]

Поизучав немного код, увидел, что где-то прописываются не используемые импорты, где-то в импортах прописаны относительные пути. В таких проектах должен быть достаточно жесткие договоренности. С python как по мне все очень просто, есть PEP8. Это мое предложение, вам его советую и жду комментариев на эту тему.

[averrin]

Я вообще большой сторонник строгого соблюдения pep (кроме длины строк), но сейчас я слегка на коленке (vim по ssh, так что пишу не очень аккуратно). Но, повторюсь, я за.

Предложение: настройте ктонить интеграцию с travis и если можно прикрутить туда проверку по pep - тогда все будут аккуратны=)

[GomZik]

@averrin по поводу длинны строк - у меня таки есть пунктик на эту тему - порой бывает нужно открыть два листинга кода рядом и ограничение в 80 символов - очень помогает :)

с travis никогда не работал, нужно посмотреть

[shmygol]

Добавлю как Rule 3.

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

[GomZik]

@trilliput ну это само собой, но желательно таки указывать на ошибку автору, чтобы он такие ошибки не плодил

[shmygol]

@GomZik Я это и имел в виду.

В идеале даже написать комментарий к коммиту автора, но это уже по возможности.

[averrin]

Добавил PEP8 в travis: https://travis-ci.org/averrin/emergent