Feature123

[IrenJones]

Validation и CS

[malinink]

@AlfredJones а как же задачу поставить?

[IrenJones]

Задача уже появилась

[malinink]

@AlfredJones Ох, откуда же столько полей в коментариях, ну да пусть будут. Он ругался на массивы? Что вы их так переформатировали? (на мой взгляд смотрятся они теперь нездорово)

[IrenJones]

Вроде ругался, но не помню точно могу попробовать вернуть в норм состояние

[malinink]

@AlfredJones Если по-хорошему, я бы порекомендовал почитать код самих ошибок и понять что иенно ему не нравится, ну и подправить, например тут:

        return view('countries/countriesIndex',['countries' => $countries]);

Ему просто не нравилось отсутствие проблема после запятой

        return view('countries/countriesIndex', ['countries' => $countries]);

В вашем же случае вышло горадо больше кода:

        return view(
            'countries/countriesIndex',
            [
            'countries' => $countries
            ]
        );

Такое форматирование иногда применяется когда агрументов много например:

        return view(
            'countries/countriesIndex',
            [
                'countries'       => $countries,
                'countries2'      => $countries,
                'countriesAndEtc' => $countries,
                'anotherArgument' => $countries,
            ]
        ); 

Обратите внимание на табы перед ключами массива, и перед значениями.

[malinink]

@AlfredJones Я понял в чем проблема, вы проверяете не совсем верно через phpcs. Видимо, вы указываете конкретное имя файла, и он проверяет не по нашему конфигу.

[IrenJones]

А не выходит, если проверить всё целиком Поэтому поодиночке проверяла

[IrenJones]

В общем, попробую разобраться

[malinink]

@AlfredJones Поглядите тут Пока специально не буду принимать PR.

[IrenJones]

Разобралась с cs Вопрос по комментированию в начале каждого: там требуется лицензия и прочие немного ненужные штуки Каждый раз придётся вставлять км коммента в файл или можно этого избежать?

[malinink]

@AlfredJones Если запускать тест phpcs верно, то сейчас нет никаких требований к коментариям(файла и класса), и они не требуются.

Мы обязательно сформируем требования в будущему проекту, среди которых будут и коментарии.

[IrenJones]

Он просто тупо виснет Если он и проверяет, то проверяет слишком медленно, не полчаса же ждать, пока он все ошибки выдаст Поэтому и приходится по одному проверять

[malinink]

Какой командой запускаете проверку?

Малинин Константин

On 12 Nov 2015, at 00:28, Iren notifications@github.com wrote:

Он просто тупо виснет Если он и проверяет, то проверяет слишком медленно, не полчаса же ждать, пока он все ошибки выдаст Поэтому и приходится по одному проверять

— Reply to this email directly or view it on GitHub.

[IrenJones]

phpcs если для всех Нахожусь в папке проекта

[malinink]

@AlfredJones команда из папки проекта один в один такая?

./vendor/bin/phpcs

Докатили последние изменения? В файле конфига закоментили строчки? У меня команда на текущем репозитории с раскоменченными строками в конфиге phpcs выполняется

Time: 785ms; Memory: 6.5Mb

[IrenJones]

Нет, не такая

Изменения докатила

Попробую вечером ещё раз

[IrenJones]

У меня были проблемы с тем, что composer без sudo не хотел update делать Но Саша помог со всем разобраться, и теперь всё работает)