Prozess: PullRequest Review Checkliste

Beinhaltet eine kurze Liste möglicher Dinge, die bei einem PR beachtet werden müssen

• Usermigration: muss nachegführt werden wenn usernamen als string gespeichert werden

• Sind UpgradeSteps vorhanden/nötig:

  • Retroaktive Veränderungen nur bei Korrekturen am Upgradestep
  • 1 Commit pro Upgradestep bzw. 1 Upgradestep pro ProfileVersion bump
  • Wurden Änderungen vorgenommen, die einen Upgrade erfordern?

• Sind UpgradeSteps deferrable, oder können gewisse Schritte des Upgrades konditional ausgeführt werden?

• Gibts es eine DB-Schema migration?

  • Wurden die Versionsnummern in der Migratisonsklasse richtig eingetragen?
  • Wurde alle Columns/Änderungen aus dem Modell in einer DB-Schema migration nachgeführt.
  • Sind constraint-namen maximal 30 Zeichen lang? (Oracle-restriktion)
  • Schema-Migrationen müssen mit PostgreSQL und MySQL getestet werden

• Könnten Kundeninstallationen von den Änderungen betroffen sein?

• Gibt es neue Funktionalität mit einem Dokument? Funktioniert das auch mit einem Mail?

• Wurde etwas an der Aufgabe angepasst? Funktioniert das auch mit einer Weiterleitung?

• Gibt es neue Übersetzungen?

  • Sind alle msg-Strings in Übersetzungen Unicode?
  • Wird die richtige i18n-domain verwendet? (Copy-Paste Fehler sind hier häufig)

• Wenn bei Schema-definitionen missing_value spezifiziert ist muss immer auch default auf den gleichen Wert gesetzt werden

• Ist die Änderung in der Trackingliste der Bugs/Features aktualisiert?

• Wird das package released?

  • Changelog-Eintrag vorhanden/nötig?
  • Package-Version angepasst? (setup.py)