matrix-org / matrix-org/mscbot-python

Users (who aren't whitelisted) who post top-level to an proposal should have their comment automatically moved to a PR review comment

Offen
#30 1 Kommentar 1 Reaktion 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

enhancement
Vorherrschende Sprache
Python
Sterne
7
Forks
2
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

The Spec Core Team often has the manual task of cleaning up comments that appear from contributors at the top-level of the MSC (a PR comment, rather than a PR review comment). Cleaning them up involves copying the contents of the comment, hiding it, then starting a new thread on the MSC (typically at line 1) and pasting the contents there as a quote. Along with some text to inform the user as to why it was necessary.

This is time-consuming manual work. And sometimes when we don't catch these comments quickly enough, other people will reply with another top-level comment, making the problem worse.

It would be nice if mscbot could take care of doing that automatically, along with a canned explanation. It should support a config option that excludes certain users (SCT, guardians, MSC author) who occasionally do need to post top-level from the correction.

Beitragsleitfaden

Beitragsleitfaden öffnen

Erste Schritte

  1. Lies das ganze Issue und danach den Beitragsleitfaden des Projekts.
  2. Schreib ins Issue, dass du es übernimmst — das erspart doppelte Arbeit.
  3. Forke das Repository und arbeite in einem Branch.
  4. Öffne einen Pull Request, der die Issue-Nummer nennt.

Rechercherichtung

Im Issue wird keine Datei, kein Test und kein Einstiegspunkt genannt. Beginne damit, den Code zu finden, der GitHub pull-request comments und die Konfiguration verarbeitet, und verfolge dann, wie Kommentare auf oberster Ebene und Ausschlüsse von Benutzern dargestellt werden. Als erledigt gilt die Aufgabe, wenn berechtigte Kommentare mit der vorgefertigten Erklärung in Review-Threads verschoben werden, während ausgeschlossene Benutzer davon nicht betroffen sind.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
github, python
Bereich
devtools
Issue-Typ
Feature
Schwierigkeit
4/5
Geschätzter Aufwand
3-5 Tage
Aktivitätsstatus
Veraltet
Klarheit
Größtenteils klar
Anfängerfreundlichkeit
35/100

Neue Issues direkt in Ihr Postfach

Eine kurze Übersicht über anfängerfreundliche GitHub-Issues.