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

migrate: pip-tools → poetry #1976

Merged
merged 109 commits into from Oct 30, 2023
Merged

migrate: pip-tools → poetry #1976

merged 109 commits into from Oct 30, 2023

Conversation

hnthh
Copy link
Member

@hnthh hnthh commented Sep 16, 2023

Что сделать после перехода:

  • Проверить входящие платежи
  • Проверить, что работает flower
  • Проверить, что работают периодические таски
  • Перенастроить branch protection, чтобы требовал шага test вместо build
  • Проверить, как renovate реагирует на замену package-manager: если обновит свои ПР — ок, если нет — позакрывать все и ждать новых
  • Починить docker-compose для фронтенда, PR тут

Что сделать до перехода:

  • исправить версию package;
  • привести все кавычки в воркфлоу-файлах к одному типу.

@hnthh hnthh added the enhancement New feature or request label Sep 16, 2023
@hnthh hnthh self-assigned this Sep 16, 2023
@f213
Copy link
Contributor

f213 commented Oct 27, 2023

В любом случае, нам стоит договориться о времени, когда мы оба будем на связи, чтобы катить твоё обновление, когда код будет готов.

Подумал получше — кажется это тот случай, когда лучше сделать пару запросов на проде. Постараюсь найти время на выходных, чтобы смёрджить всё и потестить. Думаю, это будет первая половина дня либо субботы, либо воскресенья.

@f213 f213 force-pushed the poetry branch 2 times, most recently from 7ef9a1a to 789eed0 Compare October 27, 2023 17:28
@f213
Copy link
Contributor

f213 commented Oct 28, 2023

Думаю, это будет первая половина дня либо субботы, либо воскресенья.

Сел выкатывать и понял, что с таким временем прохождения CI мы не сможем быстро чинить проблемы, поэтому свободное время потратил на оптимизацию билда.

@f213 f213 merged commit 8a1ade3 into master Oct 30, 2023
4 checks passed
@f213 f213 deleted the poetry branch October 30, 2023 15:19
@f213 f213 restored the poetry branch October 30, 2023 15:20
@hnthh
Copy link
Member Author

hnthh commented Oct 30, 2023

Уррааа! 🏄🏻‍♀️

@f213
Copy link
Contributor

f213 commented Oct 30, 2023

Спасибо тебе! Будет круто, если поможешь @nkiryanov обновить структуру приложения в #2055, чтобы ему меньше разибараться с тем, что мы тут наворотили

@hnthh
Copy link
Member Author

hnthh commented Oct 30, 2023

Будет круто, если поможешь @nkiryanov обновить структуру приложения в #2055, чтобы ему меньше разибараться с тем, что мы тут наворотили

@nkiryanov, завтра всё пофикшу ;)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants