You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Falta el README en la raíz del repo pero está en una subcarpeta; parece que en algún momento bajaron todo el repo y lo volvieron a subir en otro lado.
El proyecto compila. La funcionalidad está muy completa. La presentación fue muy buena.
Bien el uso de milestones en el repo. Bien el uso de pull request en el repo. Bien el uso de issues en el repo. Valeria tiene significativamente menos actividad que el resto en el repo.
Los test son pocos pero están bien.
Tienen excepciones que no procesan: IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs
No están siempre bien elegidos los nombres de variables: la variable "e" debería ser por ejemplo "result" en IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs
No controlan posibles resultados null: currenUser puede ser null en void OnGet() en Controller\TechnicianLayout\MyProjects.cs
Experto
En desarrollo
Incipiente
Insuficiente
Aplicas patrones de diseño
Bien
Respetas principios de diseño
Bien
Conoces un catálogo de patrones y lo usas
Bien
Programación en C#
Bien
Excepciones
No siempre controlan bien
Programa fácil de mantener, modificar, reusar
A veces mal nombres y formato
Repo
Bien
DBC y tests
Pocos pero bien
Nota: 90
The text was updated successfully, but these errors were encountered:
Falta el README en la raíz del repo pero está en una subcarpeta; parece que en algún momento bajaron todo el repo y lo volvieron a subir en otro lado.
El proyecto compila. La funcionalidad está muy completa. La presentación fue muy buena.
Bien el uso de milestones en el repo. Bien el uso de pull request en el repo. Bien el uso de issues en el repo. Valeria tiene significativamente menos actividad que el resto en el repo.
Los test son pocos pero están bien.
Tienen excepciones que no procesan: IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs
No están siempre bien elegidos los nombres de variables: la variable "e" debería ser por ejemplo "result" en IEnumerable LoadProjects() en Controller/Technicians/Details.cshtml.cs
No controlan posibles resultados null: currenUser puede ser null en void OnGet() en Controller\TechnicianLayout\MyProjects.cs
Nota: 90
The text was updated successfully, but these errors were encountered: