Skip to content

Conversation

@SanchezzRU
Copy link
Contributor

@SanchezzRU SanchezzRU commented Nov 21, 2025

Pull Request description

Changelog

  • Вынес proxy.licenseKey в секрет
  • Поправил отступы/форматирование

Issues

Breaking changes

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

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

@SanchezzRU SanchezzRU requested review from a team as code owners November 21, 2025 07:33
@SanchezzRU SanchezzRU force-pushed the fb/DEVOPS-2250-traffic-proxy-secret branch from b26d272 to abbc234 Compare November 21, 2025 07:37
@SanchezzRU SanchezzRU marked this pull request as draft November 21, 2025 07:38
@SanchezzRU SanchezzRU force-pushed the fb/DEVOPS-2250-traffic-proxy-secret branch 2 times, most recently from c5a6949 to 55d4c0c Compare November 21, 2025 08:23
@SanchezzRU SanchezzRU changed the title [DEVOPS-2250] move proxy.apiKey to secret [DEVOPS-2250] move proxy.licenseKey to secret Nov 21, 2025
@SanchezzRU SanchezzRU force-pushed the fb/DEVOPS-2250-traffic-proxy-secret branch from 66dad89 to dc79c56 Compare November 21, 2025 10:11
@SanchezzRU SanchezzRU marked this pull request as ready for review November 21, 2025 10:18
@SergeyKodolov
Copy link
Collaborator

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

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.

3 participants