envoyproxy/envoy

RepoKitteh-based PR checklists

Open

#12.949 geöffnet am 2. Sept. 2020

Auf GitHub ansehen
 (13 Kommentare) (0 Reaktionen) (1 zugewiesene Person)C++ (5.373 Forks)batch import
area/repokittehbeginnerenhancementhelp wanted

Repository-Metriken

Stars
 (27.997 Stars)
PR-Merge-Metriken
 (Durchschn. Merge 8T) (378 gemergte PRs in 30 T)

Beschreibung

We can have RK automatically label any change to source/ or api/ without a change to current.rst with a release-notes blocker, requiring manual confirmation from maintainers that the release notes step has been satisfied. Similarly this could work for documentation checks, i.e. validate that at least some change has happened in docs/.

I'd propose that we take the existing ownerscheck.star and add a path expression that suppresses a given check. That should be all that is needed for release notes or docs.

@envoyproxy/maintainers are there other common review checklist items that RK could help with? CC @itayd

Contributor Guide

Research-Richtung
Untersuchen Sie die vorhandene Datei ownerscheck.star im Repository, verstehen Sie, wie Pfadausdrücke funktionieren, und fügen Sie eine neue Pfadunterdrückung für Versionshinweise und Dokumentationsprüfungen hinzu.
Tech Stack
Keine
Domain
devops
Issue Type
Funktion
SchwierigkeitGeschätzte Implementierungsschwierigkeit für neue Contributors, von 1 für sehr kleine Änderungen bis 5 für Expertenarbeit.
3
Geschätzte ZeitEin grober Zeitrahmen, um zu recherchieren, zu implementieren, zu testen und einen Pull Request vorzubereiten.
Halber Tag
AktivitätsstatusWie verfügbar das Issue gerade wirkt: frisch, aktiv, stale, blockiert oder wartet auf Maintainer-Input.
Aktiv
KlarheitWie klar das Issue die erwartete Änderung, Akzeptanzkriterien und den nächsten Schritt erklärt.
Klar
Voraussetzungen
Git
EinsteigerfreundlichkeitEin Score von 1 bis 100, der schätzt, wie zugänglich dieses Issue für First-Time Contributors ist.
65