Skip to content

Conversation

@i-bogomazov
Copy link
Contributor

@i-bogomazov i-bogomazov commented Jan 19, 2025

Pull Request description

Changelog

Issues

  • n/a

Breaking changes

  • n/a

Check-list. Чек-лист код-ревью

  • Запрос на слияние в develop.
  • Есть описание к PR.
  • Указаны блокирующие изменения. Breaking-Changes
  • Соответствие кода принятому стилю
    • Описание настроек.
    • Именование настроек.
    • Дефолтные значения.
    • Стиль кода.
  • Работоспособность. Разворачивается на своем окружении из ветки PR.
    • Тест API через тесты helmfile-хуков или коллекций Postman.
  • Не осталось мусора от удаления каких-то параметров. Ищется поиском по проекту из ветки PR.
  • Отработка линтера на чарт из ветки PR. Пример: helm lint charts/search-api

@i-bogomazov i-bogomazov requested review from a team as code owners January 19, 2025 10:35
@dbelyaev-nsk
Copy link
Contributor

задача на проверку https://jira.2gis.ru/browse/ONPREM-1414

@mprudnikov3
Copy link
Contributor

@i-bogomazov, необходимо вынести параметры:

@i-bogomazov i-bogomazov force-pushed the fb/navi-restrictions-1.0.2 branch from f4742ba to 5bed53b Compare April 8, 2025 08:37
@i-bogomazov
Copy link
Contributor Author

@i-bogomazov, необходимо вынести параметры:

* `LOG_JSON`

* `LOG_LEVEL`
  Согласно https://tsup.2gis.dev/navi/restrictions/1.0.2

сделано

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants