Ops247

[FalkoBenthin]

Checkliste

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

Allgemein

• Links zu verwandten PRs anderer Repositories
  • travis should fail if npm audit finds security vulnerabilities with s… schulcloud-client#1557
  • Im Ticket (oder PR, wenn kein Ticket): Beschreibung und Begründung der Änderung/Neuerungen
• 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?

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
  • Swagger
  • Confluence - https://docs.schul-cloud.org
  • Guidelines / Hilfe
  • Release Notes - wenn es sich um große Feature-Releases handelt

Datenschutz

• [ ] Model- / Seedänderungen erfordern Review der Datenschutz-Gruppen (wird automatisch hinzugefügt)
• [ ] 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
• WIP PR-Label entfernt, wenn die Checkliste abgearbeitet wurde

Mehr

Weitere Informationen zur DoD hier im Confluence

[codecov]

Codecov Report

❗ No coverage uploaded for pull request base (develop@8c052a1). Click here to learn what that means.
The diff coverage is 71.53%.

@@            Coverage Diff             @@
##             develop     #921   +/-   ##
==========================================
  Coverage           ?   66.94%           
==========================================
  Files              ?      258           
  Lines              ?     8036           
  Branches           ?        0           
==========================================
  Hits               ?     5380           
  Misses             ?     2656           
  Partials           ?        0

Impacted Files	Coverage Δ
src/services/fileStorage/utils/copyFile.js	24.32% <ø> (ø)
src/services/clipboard/index.js	100% <ø> (ø)
src/sockets/clipboard/clipboard-model.js	100% <ø> (ø)
src/services/homework/model-service.js	100% <ø> (ø)
src/services/sync/strategies/SystemSyncer.js	28.57% <0%> (ø)
src/services/releases/index.js	68.42% <0%> (ø)
src/services/sync/strategies/LDAPSyncer.js	10.25% <0%> (ø)
src/services/statistic/index.js	46.34% <0%> (ø)
src/services/ldap/strategies/general.js	5.55% <0%> (ø)
src/services/ldap/index.js	13.82% <0%> (ø)
... and 206 more

---

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 8c052a1...8cb9e6d. Read the comment docs.