David-S-IT / yadjango

Образовательный проект по Django от Интенсивы Академии Яндекса
1 stars 1 forks source link

Не забываем проверять терминал CI/CD на гитхабе #17

Closed drowsycoder closed 1 year ago

drowsycoder commented 1 year ago

Меня сильно смущает фраза

No Python files are present to be formatted. Nothing to do

в black Lint > Black Check

Возможно, это затронет тот момент, что придётся как-то спасать сворачивание строк в AUTH_PASSWORD_VALIDATORS в settings.py и аналогичные им случаи

Для примера, если зайти в воркфлоу тестирования, там ок:

Ran 14 tests in 0.022s

(Только сам бейдж хочется назвать как-то типа Django Tests)

David-S-IT commented 1 year ago

Не совсем почему вас смущает фраза

No Python files are present to be formatted. Nothing to do

Поясните это, пожалуйста.

drowsycoder commented 1 year ago

Локально до пуша запускаешь black? black --check --verbose --diff --skip-string-normalization --exclude 'venv/' .

David-S-IT commented 1 year ago

Нет) только flake8

David-S-IT commented 1 year ago

Он жалуется на то, что нельзя исправлять.. Не могли бы Вы, пожалуйста, подсказать, что делать?

(venv) dav@dav-GE60-2PE:~/Dev/yadjango$ black --check --diff .
--- /home/dav/Dev/yadjango/ya/ya/settings.py    2023-02-15 17:19:33.360412 +0000
+++ /home/dav/Dev/yadjango/ya/ya/settings.py    2023-02-15 17:19:53.613920 +0000
@@ -39,13 +39,11 @@
     'django.contrib.auth',
     'django.contrib.contenttypes',
     'django.contrib.sessions',
     'django.contrib.messages',
     'django.contrib.staticfiles',
-
     'debug_toolbar',
-
     'about.apps.AboutConfig',
     'catalog.apps.CatalogConfig',
     'homepage.apps.HomepageConfig',
 ]

@@ -101,23 +99,20 @@
 # https://docs.djangoproject.com/en/3.2/ref/settings/#auth-password-validators

 AUTH_PASSWORD_VALIDATORS = [
     {
         'NAME': 'django.contrib.auth.'
-                'password_validation.UserAttributeSimilarityValidator',
+        'password_validation.UserAttributeSimilarityValidator',
     },
     {
-        'NAME': 'django.contrib.auth.'
-                'password_validation.MinimumLengthValidator',
+        'NAME': 'django.contrib.auth.' 'password_validation.MinimumLengthValidator',
     },
     {
-        'NAME': 'django.contrib.auth.'
-                'password_validation.CommonPasswordValidator',
+        'NAME': 'django.contrib.auth.' 'password_validation.CommonPasswordValidator',
     },
     {
-        'NAME': 'django.contrib.auth.'
-                'password_validation.NumericPasswordValidator',
+        'NAME': 'django.contrib.auth.' 'password_validation.NumericPasswordValidator',
     },
 ]

 # Internationalization
 # https://docs.djangoproject.com/en/3.2/topics/i18n/
would reformat /home/dav/Dev/yadjango/ya/ya/settings.py

Oh no! 💥 💔 💥
1 file would be reformatted, 31 files would be left unchanged.
drowsycoder commented 1 year ago

Если не менять конфиг:

  1. Поставь комменты с указанием разделов
  2. {
        'NAME': (
            'django.contrib.auth'
            '.password_validation.UserAttributeSimilarityValidator'
        ),
    },
    {
        'NAME': (
            'django.contrib.auth.password_validation.MinimumLengthValidator'
        ),
    },

    А так внимательнее ознакомься с правилами black, когда, где и как их применять, как создавать исключения Заодно проверь, всё ли верно с line-lenght=79

А вместо

No Python files are present to be formatted. Nothing to do 😴

на гитхабе должна быть строчка типа такой:

All done! ✨ 🍰 ✨
36 files would be left unchanged.
David-S-IT commented 1 year ago
  1. Поясните, пожалуйста, что вы подразумеваете под комментами?
  2. Я line-lenght=79 поставил, а всё равно
    [tool.black]
    line-lenght=79
    skip-string-normalization=true

    написал в файле pyproject.toml

drowsycoder commented 1 year ago
  1. Вместо пустых строк, если ты не обрабатываешь это правило, можешь поставить строки, в которых комментарии, касающиеся последующих строк
  2. Ну, во-первых, правописание слова lenght надо проверить, посмотреть подробный вывод в терминале после выполнения команды (для подробного вывода смотрим её ключи для командной строки), посмотреть конфиг, почитать доки об этом и других ключах и это проанализировать

Серьёзно, я в тебя верю, ты сможешь Иначе, если не научишься искать выход из простых ситуаций, уже через несколько лекций будет совсем тяжело