feat(ci): skrzynka akceptacji - na maila trafia tylko robota skonczona - #171
Merged
Merged
Conversation
Two faults, one fix. The discovery bots assigned every raw candidate to the owner, so the inbox filled with things that were not ready. Reviewing that first queue found eight leads and zero actionable ones: five threads closed and resolved, two dead since 2022, and one the owner had already answered six days earlier. An inbox of unfinished work is an inbox that stops being read. And none of it notified anyway. GitHub suppresses notifications for the actor who performed the action, so assigning the owner with the owner's own token emailed nobody. Verified against the event log: actor msgwing, assignee msgwing. So the bots no longer assign. They queue, labelled, and that is all. Labelling an issue `do-akceptacji` once it is genuinely finished hands it over: ready-for-approval.yml assigns the owner - as github-actions[bot], a different actor, so the mail actually goes out - and posts the instruction in Polish, because the owner is the only person reading it. The payload inside each issue stays in English. It is going to be pasted into an English thread or a directory entry, and translating it would mean translating it back. Rejecting is a first-class path: remove the label and say what is wrong, rather than fixing it by hand. The correction then lands where the fault was, instead of in one issue that gets fixed and forgotten.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Dwie usterki, jedna poprawka.
Problem 1 — na maila szło niedokończone
Boty przypisywały właścicielowi każdego surowego kandydata. Przegląd tej pierwszej kolejki dał osiem leadów i zero nadających się do działania: pięć wątków zamkniętych i rozwiązanych, dwa martwe od 2022, jeden już odpowiedziany przez właściciela sześć dni wcześniej. Skrzynka pełna niedokończonej roboty to skrzynka, którą przestaje się czytać.
Problem 2 — i tak nic nie wychodziło
GitHub nie powiadamia o działaniach, które sam wykonałeś. Przypisanie właściciela jego własnym tokenem nie wysyła nic. Zweryfikowane w dzienniku zdarzeń:
aktor: msgwing, przypisany: msgwing.Rozwiązanie
Boty przestają przypisywać. Kolejkują, etykietują i tyle — bez maila.
Nadanie etykiety
do-akceptacjina skompletowanej sprawie przekazuje ją dalej:ready-for-approval.ymlprzypisuje właściciela jakogithub-actions[bot], czyli inny aktor, więc mail faktycznie wychodzi — i dokleja instrukcję po polsku.Języki
Instrukcja po polsku, bo czytasz ją tylko Ty. Treść do publikacji zostaje po angielsku — pójdzie do angielskiego wątku albo wpisu w katalogu, więc tłumaczenie znaczyłoby tłumaczenie z powrotem.
Odrzucenie jest równoprawną ścieżką
Jeśli coś nie gra — nie poprawiasz ręcznie. Zdejmujesz etykietę i piszesz w komentarzu, co jest nie tak. Poprawka trafia tam, gdzie powstał błąd, zamiast w jedno zgłoszenie, które zostanie naprawione i zapomniane.