paladin223 / django

0 stars 0 forks source link

Можно чуть дооптимизировать CI/CD #4

Closed drowsycoder closed 1 year ago

drowsycoder commented 1 year ago

Версию python-version можно оставить только одну

(В остальном файл должен, наоборот, расшириться, если будешь делать второй допник, он же Задача № 3)

И не стоит указывать там --ignore=F401 для флейка. Это, скорее, в файлы настроек надо (всякие там setup.cfg и т.п.) Но в идеале — минимум игноров

(А саму проверку проекта у себя при разработке в любом случае стоит расширить за счёт плагинов Примерно оптимальная установка флейка с плагинами для базовых вещей: pip install flake8 pep8-naming flake8-broken-line flake8-return flake8-isort flake8-quotes Соответственно, фризим зависимости, используем их в CI/CD как файл, а по терминалу гитхаб экшнс смотрим, что не пытаемся их ставить повторно)

drowsycoder commented 1 year ago

Лучше вынести black в какой-нибудь отдельный раздел CI/CD типа Check with black

drowsycoder commented 1 year ago

Не доработано Lint with flake8 and black: шаги стоит разделить для флейка и блэка