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

Release/timecodes #41

Open
wants to merge 17 commits into
base: release/payments-api
Choose a base branch
from
Open

Release/timecodes #41

wants to merge 17 commits into from

Conversation

AshurovG
Copy link
Collaborator

@AshurovG AshurovG commented Dec 9, 2024

Задача №00000000

Что было не так:

<описание проблемы/причины заведения новой фичи>

Что было сделано:

  • слой repository;
  • добавлена валидация на уникальность email;
  • ручка на получение подборок фильмов;
  • написаны тесты и моки.

Чек лист до ревью :

  • PR разбит на логические коммиты (если нет, то нужно разбить);
  • PR не содержит секретов (пароли, токены, ключи);
  • PR не содержит лишних файлов (например, .env);
  • PR не содержит коммитов, которые не относятся к задаче;

Перед тем, как отдать на ревью нужно убедиться, что все пункты выполнены

как вливать ветку

Если это обычная задача, то просто squash merge в develop
Если это хотфикс, то обычный merge в master и develop (должно быть два pull request). При этом в ветке должен быть один коммит
Если это релиз, то обычный merge и в master и в develop. Там будет несколько коммитов

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.

2 participants