Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/postgres local stage 2 #72

Merged
merged 8 commits into from
Dec 20, 2023

Conversation

blizard-sis
Copy link

Description

переделал makefile и подправил docker-compose.dev.yaml
обновил зависимости develop.txt и изменил settings.py

для запуска необходимо скопировать .env.example в .env чтобы запустить проект
вначале make run-db для запуска контейнера с базой
после можно make run

Type of change

Пожалуйста, удалите варианты, которые не относятся к ПР-у.

  • Documentation (опечатки, примеры кода или любое обновление документации)
  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

How Has This Been Tested?

обкатал на маке make run-db и make run. контейнер стартует и висит в активных, можно спокойно перезапускать проект сколько угодно.
получение переменных окружения проверено

Checklist:

  • Мой код соответствует code-style данного проекта
  • Я провел самоанализ собственного кода
  • Я внес соответствующие изменения в документацию

@blizard-sis blizard-sis requested review from Raidzin and removed request for Raidzin December 19, 2023 06:44
Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Я тут разрешил конфликты, но таску добей пожалуйста в следующем ключе:
Есть новая версия docker compose, и вызывается docker compose
Ну а у кого то старые версии вызывается docker-compose
В общем тут пример https://github.com/Studio-Yandex-Practicum/RandomCoffeeBotTelegram/blob/develop/Makefile

@blizard-sis
Copy link
Author

а я должен был сам разрешить конфликт? просто когда был модуль с общей работой это делал тимлид
переделал согласно примеру и проверил опять на своей системе

Copy link
Member

@KonstantinRaikhert KonstantinRaikhert left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Всё работает, мержим👍

@KonstantinRaikhert KonstantinRaikhert merged commit 10754a8 into dev Dec 20, 2023
6 checks passed
@KonstantinRaikhert KonstantinRaikhert deleted the feature/postgres-local-stage-2 branch September 20, 2024 13:45
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.

Добавить postgres для локальной и stage разработки
2 participants