Skip to content

Comments

Обновляем работу PR и Issue на Github#14468

Open
AzzyDreemurr13 wants to merge 24 commits intoTauCetiStation:masterfrom
AzzyDreemurr13:Issue-PR-upd
Open

Обновляем работу PR и Issue на Github#14468
AzzyDreemurr13 wants to merge 24 commits intoTauCetiStation:masterfrom
AzzyDreemurr13:Issue-PR-upd

Conversation

@AzzyDreemurr13
Copy link
Contributor

@AzzyDreemurr13 AzzyDreemurr13 commented Feb 8, 2026

Описание изменений

Так как GitHub ввел новую функцию с "Типом" issue, обновляем наши с новыми возможностями и лучшей сортировкой issue.
Починка бота когда тот не выдавал проблемным ПРам автоматически Merge conflict
Pull Request - теперь если кто-то из кодеров забыл проставить в чеинжлоге что меняет данный ПР или чеинжлог не требуется, то теперь бот Гитхаба будет проверять что за файлы были изменены и выставлять соответствующие значки ПРам. (Устраняем пустые ПРы которые были без значков и с первого взгляда не понять, что они меняли если искать по ПРам на гите)

Однако, требуется помощь @TauCetiStation/maintainers , в создании лишь одного label -> Code
Code будет использоваться за место Bug в новых issue, так как они будут отмечены новым типом "Bug", в Code даст понять что проблема с механом, функцией и т.п. а не с переводом, картой и т.д.
Для Pull Request оно будет отмечать если были затронуты файлы тип .dm

Полигон для тестов: https://github.com/TauCetiStation-TranslationDepartment/TauCetiClassicPRtest/issues

Почему и что этот ПР улучшит

  1. Качество issue и Pull Request
  2. Более четкое понятие, где именно проблема и для мейнтейнеров будет видно, что приблизительно уже менялось в ПРе и есть ли в нем конфликты

Авторство

Код взят: CeladonSS13/Shiptest#2133
Перенос кода и адаптация: Я

Чеинжлог

Не требуется, обновление для GitHub TauCeti

@TauKitty
Copy link
Contributor

TauKitty commented Feb 8, 2026

Changelog status: 🔘 No changelog

* Upd PRs

* conflict-fix

* Bug-issue-upd

* Заканчиваем с обновлением

* Нашел запятую
* Upd PRs

* conflict-fix

* Bug-issue-upd

* Заканчиваем с обновлением

* Нашел запятую

* чиним 🧠 (значок localisation)
@AzzyDreemurr13
Copy link
Contributor Author

Можно здесь протестировать изменения:
https://github.com/TauCetiStation-TranslationDepartment/TauCetiClassicPRtest/issues

* Upd PRs

* conflict-fix

* Bug-issue-upd

* Заканчиваем с обновлением

* Нашел запятую

* чиним 🧠 (значок localisation)

* fix3
* Upd PRs

* conflict-fix

* Bug-issue-upd

* Заканчиваем с обновлением

* Нашел запятую

* чиним 🧠 (значок localisation)

* fix3
* Issue pr upd (TauCetiStation#6)

* Upd PRs

* conflict-fix

* Bug-issue-upd

* Заканчиваем с обновлением

* Нашел запятую

* чиним 🧠 (значок localisation)

* fix3

* Fix translation tools section in LABELER.yml
@@ -1,11 +1,23 @@
name: "Сообщить о проблеме"
description: "Если что-то работает не так, как ожидалось."
labels: ["Bug"]
Copy link
Member

Choose a reason for hiding this comment

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

Я вот хзхз. Из минусов, надо в голове помнить, что старые баги были под ЛЕЙБЛОМ "баг", а под ТИПОМ "баг" новые.

Мне просто чет кажется, лучше баг-лейбл оставить на месте

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Ответил в Дискорде

Co-authored-by: KIBORG04 <bossmira4@gmail.com>
@TauKitty TauKitty requested a review from a team February 16, 2026 17:45
runs-on: ubuntu-latest
steps:
- name: Check for conflicting PRs
uses: eps1lon/actions-label-merge-conflict@releases/2.x
Copy link
Member

Choose a reason for hiding this comment

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

Еще из неочевидных моментов, у нас вроде на репозитории включены защиты всякие, которые блокируют запуск сторонних экшенов. Так что... Тут чисто к воласу, он это пропускает или запрещает

Comment on lines +48 to +49
- 'config/**/*'
- 'config/example/**/*'
Copy link
Member

Choose a reason for hiding this comment

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

Suggested change
- 'config/**/*'
- 'config/example/**/*'
- 'config/**/*'

Он же сам по первому условию подтянет? Я если что не уверен.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Да, подтянет

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Хотя, я может и не прав, на отдельном форке гляну

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Оно из-за гитигнора, может не сработать. Так что возвращаю.
image

Co-authored-by: KIBORG04 <bossmira4@gmail.com>
@TauKitty TauKitty requested a review from a team February 16, 2026 18:07
AzzyDreemurr13 and others added 5 commits February 16, 2026 21:08
Для LABELER
- json файлы получают лейб javascript
- Папка Tools вынесена с Maintainaility в Tools
- Добавил Переводческий вики файл под лейб Translation
- html и nano файлы также будут получать UI
- В конфиг апдейты добавил примеры из-за гитигнора

Bug_report.yml
- Вынес Что должно произойти и что произошло в отдельные графы
- Добавил инфу про баг репорт в углу на сервере
- Добавил обязательный выбор для выбираемого типа ишуя
Не работает с checkboxes, грустно, но не критично
@AzzyDreemurr13 AzzyDreemurr13 requested a review from volas February 18, 2026 22:32
@AzzyDreemurr13
Copy link
Contributor Author

Вроде добавил все что хотел и то что обсуждал с Киборгом, остальное осталось по организационным вопросам... что можно и нельзя, ну и возможно что переделать

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants