Testng: Определить / использовать соглашение о коде для нового / обновленного кода

Созданный на 4 мар. 2017  ·  8Комментарии  ·  Источник: cbeust/testng

Я думаю, мы должны определить соглашение о коде для нового и обновленного кода, чтобы везде иметь одинаковую конфигурацию IDE и избегать опечаток в синтаксисе в запросах на вытягивание.

Обычно я использую соглашение Google, потому что они предоставляют идею и файл конфигурации eclipse .

@ testng-team WDYT?

Question perenhancement

Все 8 Комментарий

Я согласен. Мы могли бы также включить файл editorConfig . В случае средства форматирования и пользователю все равно потребуется импортировать его в свою среду IDE, чтобы он вступил в силу. Но в случае editorConfig он идет в комплекте с IntelliJ. Пожалуйста, смотрите полный список здесь
У них также есть плагин для eclipse, который можно скачать. Я не пытаюсь сравнивать это с этим, а просто выбрасываю варианты.

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

Мы можем обсуждать правила, но я не хочу тратить много времени, потому что не хочу следовать правилам, которые мне не нравятся (после того, как они определены).
Соглашение Google могло бы стать хорошим началом, и мы можем настроить его в соответствии с мнением @ testng-team.

Согласен, было бы неплохо включить в проект правила форматирования для IDEA и Eclipse, но, по моему опыту, это не всегда легко без указания предпочтений для многих других вещей в среде IDE (не уверен, что это все еще верно). Если возможно зафиксировать файл, в котором указывается только форматирование кода и ничего больше, отлично.

Кроме того, этот файл в идеале должен определять порядки импорта, чтобы мы никогда не видели перетасовку импорта в PR.

@cbeust А WDYT про гугл-стиль?

Меня устраивает стиль Google.

Согласен, было бы неплохо включить в проект правила форматирования для IDEA и Eclipse, но, по моему опыту, это не всегда легко без указания предпочтений для многих других вещей в среде IDE (не уверен, что это все еще верно).

Это все еще отстой сегодня, для порядка импорта eclipse имеет отдельное предпочтение и может быть экспортировано.
На данный момент наша команда использует как Eclipse, так и Intellij, мы используем предпочтения eclipse как источник истины. Для Intellij существует плагин Eclipse Code Formatter, который может импортировать файлы настроек eclipse (включая порядок импорта)

@missedone По предложению @krmahadevan ,

Кстати, если мы будем следовать стилю Google, они уже предоставляют файлы конфигурации eclipse и intellij.

Раньше я не использовал editorConfig, было бы весело попробовать что-то новое, но похоже, что нужно определить профиль editorConfig с нуля, чтобы он соответствовал TestNG ...

Я бы сказал, давайте сначала начнем со стиля Google.
для заказа на импорт должен быть отдельный файл профиля.

Была ли эта страница полезной?
0 / 5 - 0 рейтинги