travis should fail if npm audit finds security vulnerabilities with s…

[FalkoBenthin]

…everity high or higher

Checkliste

• Bis die komplette Checkliste abgearbeitet ist muss das PR-Label WIP gesetzt sein

Allgemein

• [ ] Links zu verwandten PRs anderer Repositories
  • https://github.com/schul-cloud/schulcloud-server/pull/????
  • Im Ticket (oder PR, wenn kein Ticket): Beschreibung und Begründung der Änderung/Neuerungen
• [ x ] Link zum Ticket https://ticketsystem.schul-cloud.org/browse/OPS-247

Code Qualität

• [ ] Code mit Hinblick auf Security und Datensicherheit betrachten
• [ ] Linter darf keine Probleme bei veränderten Dateien aufweisen
• [ ] Kern-Logik ist hinter der API implementiert?

UX

• [ ] UI-Änderungen wurden von der UX-Gruppe akzeptiert

Tests

• [ ] Test-Coverage darf durch PR nicht sinken
• [ ] Unit-Tests und Integrations-Tests schreiben / ändern
• [ ] Keine offenen bekannten Bugs im entwickelten Code

Deployable

• [ ] Feature Toggle notwendig (z.B. Environment-Variablen)
• [ ] Datenbankanpassungen notwendig?
  • Gibt es ein Migrationsskripte?
  • Alle DB-Anpassungen müssen in den Seed-Daten reflektiert werden
• [ ] Notwendige neue Konfiguration an der Infrastruktur wurde mit Dev-Ops besprochen

Dokumentation

• [ ] Neue Abhängigkeiten (Repos, NPM Pakete, Vendor Skripte) begründen und überprüfen (Stabilität, Performance, Aktualität, Autor)
  • Begründung:
• [ ] Übergabe/Schulung & Administrationsinfos (#Busfaktor, Confluence intern)
• [ ] Dokumentation (wenn notwendig)
  • Code
  • Confluence - https://docs.schul-cloud.org
  • Guidelines / Hilfe
  • Release Notes - wenn es sich um große Feature-Releases handelt
• [ ] mind. 1 Screenshot bei Content-Änderungen

Datenschutz

• [ ] Neue Verarbeitung von personenbezogene Daten wurde mit der Datenschutz-Gruppe besprochen

Freigabe zum Review

• [ ] Die Änderungen wurden mit dem Ticket-Ersteller, Support-Team oder PO besprochen und erfüllen die Ticket-Anforderungen
• [ x ] WIP PR-Label entfernt, wenn die Checkliste abgearbeitet wurde

Mehr

Weitere Informationen zur DoD hier im Confluence

[pullapprove]

Hey @FalkoBenthin, thanks for the PR! The review will start once
the tests, CI checks and PR requirements (see checklist in your PR) have passed.