helm / helm/pull-sizer

Including tests+docs in sizing discourages both

Offen
#9 0 Kommentare 0 Reaktionen 0 zugewiesene Personen Auf GitHub ansehen

Dieses Issue hat noch niemand übernommen.

Vorherrschende Sprache
Go
Sterne
8
Forks
6
PR-Merge-Kennzahlen
Keine gemergten PRs in 30 T.

Beschreibung

I’ve found on multiple occasions that adding unit tests or docs to a PR (or a PR that’s literally just unit tests) almost instantly bumps the PR into needing a +2 because of the size.

While the team is awesome about this, it seems like this almost discourages writing docs and tests.

I doubt there’s any appetite for changing how the bot does its calculations, but I thought I’d raise the topic in the hopes that maybe someone could educate me as to the benefits / rationale for the current method, or maybe it would stimulate some conversation about alternate strategies.

Thoughts?

Beitragsleitfaden

Für dieses Repository ist kein Beitragsleitfaden indexiert

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

Beginne mit der Überprüfung der aktuellen Regeln des pull-sizer-Bots zur Größenberechnung, insbesondere wie Unit-Tests und Änderungen an der Dokumentation gezählt werden. Untersuche die Begründung für die aktuelle Methode und vergleiche mögliche alternative Strategien; das Issue ist erst abgeschlossen, wenn ein konkreter Ansatz vereinbart und spezifiziert wurde.

Vom Indexierungsmodell aus dem Issue-Text verfasst.

Bewertung

Tech-Stack
go
Bereich
tooling
Issue-Typ
Feature
Schwierigkeit
5/5
Geschätzter Aufwand
Über eine Woche
Aktivitätsstatus
Veraltet
Klarheit
Muss geklärt werden
Anfängerfreundlichkeit
25/100

Neue Issues direkt in Ihr Postfach

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