Projekt

Obecné

Profil

Hodnocení 5. iterace

Přidáno uživatelem Premek Brada před téměř 4 roky(ů)

Hodnocení

10 bodů

26.5.2021

Doporučení

  • vyjasnit se zadavatelem licenci k vytvořenému sw (open source použitý v CATV), licence by optimálně měla být uvedena jako dokument aspoň v kořenovém adresáři

Průběh a stav projektu

  • dokončena funkčnost
  • příští iterace na začištění projektu a předání (ETA < 4.6.)
pozitiva
  • projekt běží hladce
negativa
  • žádné beta/field testování a test nasazení, oboje vede na (do značné míry neřešené) mírné riziko problémů na poslední chvíli, podrobněji viz níže
další komentáře
  • dosud strávený čas: 234 hodin

Hodnocení: dobré (3b)

Iterace

  • cíl a obsah odpovídá fázi procesu
  • opět trochu prodloužena (4 dny) - zákazník nemohl na původní demo, dodělávání věcí které se nestihly
pozitiva
  • úpravy UI dle customer demo => potřeb zákazníka
negativa
  • burndown peak 12.5. daný pozdě vytvořeným issue s podstřeleným odhadem pracnosti na refactor (Krýsl), ale hodně issues uzavřeno až na konci iterace

Hodnocení: dobré(3b)

Technická kvalita

  • kód a git pěkný, ostatní součásti jen ucházející
pozitiva
  • přehledný a komentovaný src kód
  • tag+release na výsledek iterace
  • poměrně systematické použití feature/enh/task (nastaveno minule)
negativa
  • issues bez rozlišení priorit, schema feature/enh/task s výpadky -- např #8668 Admin panel nemá být Task ale Feature (viz podřízená issues)
  • v doc arch nebyly opraveny minule diskutované body, není pravda že je použit IoC kontejner jak popisováno (třída Container je pro třídy pluginu spíše brokerem/fasádou)
  • nejsou výsledky testů, viz postupy

Hodnocení: slušné (2b)

Postupy a praktiky

  • celkově bez větších problémů
  • beta/field testy nebyly ale na každém iteration demo procházeli UI tak jak je zákazník vedl, tj. snad OK
pozitiva
  • github konvence dle GitFlow fungují (viz i retrospektiva)
  • odhady pracnosti vcelku sedí se spent time
negativa
  • testy nepoužity z nedostatku času (přepracování UI), natož jako pre-commit jak plánováno minule
  • neproveden pokus o analýzu rizik spojených s nasazením: předpokládají nasazení až naostro (s tech.konzultantem J.Vaňkem) = potenciální riziko, v rámci toho upgrade PHP 7.3 na 7.4 na serveru = potenciální riziko, diskutováno (v případě selhání udělají downgrade pluginu na PHP 7.3)

Hodnocení: slušné (2b)


Komentáře