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

Merge Upstream Workflow pt. 2 #1709

Merged

Conversation

m-dzianishchyts
Copy link
Collaborator

Что этот PR делает

Улучшения мерж воркфлоу.

  • Перевод чейнджлога через GPT 4o.
  • Проверка на существование мерж ПРа. Чья-то работа не будет случайно сброшена форс-пушем.
  • Отключение буферизации вывода логов.
  • Использование GitHub App в проверке чейнджлога (в крысу).

Тестирование

Проверил на форке.

@m-dzianishchyts m-dzianishchyts added the 📜 CL не требуется Эти изменения не влияют на игровой процесс или игроки по какой-то причине не должны о них знать label Dec 23, 2024
@github-actions github-actions bot added :shipit: Улучшение Наконец-то это сделали удобнее и лучше... 🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. labels Dec 23, 2024
Copy link

@Furrior Furrior left a comment

Choose a reason for hiding this comment

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

Раскинь везде, где не хватает, тайп хинтов заодно

tools/merge-upstream/translation_context.txt Outdated Show resolved Hide resolved
tools/merge-upstream/translation_context.txt Outdated Show resolved Hide resolved
tools/merge-upstream/translation_context.txt Outdated Show resolved Hide resolved
Copy link

@Furrior Furrior left a comment

Choose a reason for hiding this comment

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

Особо внимательно не смотрел, но, вроде, норм

@m-dzianishchyts m-dzianishchyts merged commit 1a8b92c into ss220club:master Dec 25, 2024
15 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🎸 Инструменты Мы выдаем себя за реальное сообщество разработчиков. 📜 CL не требуется Эти изменения не влияют на игровой процесс или игроки по какой-то причине не должны о них знать :shipit: Улучшение Наконец-то это сделали удобнее и лучше...
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants