Skip to content

Contribution hints

  • Der MR für "External Contributions" wird nie wieder erwähnt. Das wirkt ein wenig wie: Ja, mach mal einen Beitrag. Im Folgenden liest du dann, wie dieser nicht beachtet wird. (Ja, polemisch... aber das könnte nochmal expliziter aufgenommen werden. Z.B. durch "Integrate the changes form the MR manually".)
  • Der Satz "We will integrate your changes and close the merge request. In addition, we add your name to the list of contributors." verspricht sehr viel, das wir ggf. nicht einhalten wollen... Hacktober etc.
  • Dem Skript fehlt an den entscheidenen Stellen die Dokumentation (Konfigurationsmöglichkeiten am Anfang, Zeilen 5-20).
  • Der Hinweis den alten (ersetzten) Commit zu droppen scheint mir viel wichtiger zu sein, als es im Moment dargestellt wird (mitten in einem Code-Segment...)
  • Am Ende könnte man nochmal die GC laufen lassen, damit die alten Commits auch wirklich weg sind.

Merge request reports