datauy/ElijoEstudiar

Revisión de evaluación técnica.

Crusiris opened this issue · 5 comments

Crusiris commented

Este issue forma parte de un procesos de revisión de la piloto @BID-COMUNIDADES/laboratoria-code-squads de la iniciativa Código para el desarrollo de @EL-BID.

Issue de revisión de evaluación técnica:

Al haber contrastado la información presentadas en el repo oficial con los campos solicitados en la plantilla de README.md hemos encontrado los siguiente puntos a mejorar:

  • Agregar el microservicio de reportes de evaluación estática de código de Sonarcloud, teniendo en cuenta que las condiciones ideales de los reportes son:

          1 Ausencia de fallos estructurales
      2 Menos del 25% de líneas duplicadas
      3 Menos de 10 problemas críticos
      4 Más del 50% de clases, interfaces y métodos documentados
      5 Deuda técnica menor a 30 días
      6 Cobertura de test más del 25%

  • Agregar el badge de este servicio visible en el readme.md.

  • Agregar el microservicio de integración continua de código (opcional).

  • Agregar el badge de este servicio visible en el readme.md (opcional).

  • Actualizar versión de todas las dependencias.

bid-code4dev commented

Hola @danielcarranza nos avisan si tienen preguntas sobre el uso de sonarcloud. Es el punto obligatorio para la evaluación técnica.
Pueden registrarse gratuitamente con su cuenta de github y vincular este repo con ese servicio y solo les tomará unos minutos para poder visibilizar el reporte del último commit. Luego de eso les pedimos que como indican las laboratorians agreguen el badge de este servicio en el readme.md. Quedamos atentos. gracias!

danielcarranza commented

Hola @knlopenknowledge, quedó claro y ya lo activamos para los tres proyectos. Sin embargo nos da error con éste y por eso todavía no pasamos el enlace/badge de SonarCloud.
Estamos pidiendo ayuda con el error en los foros de SonarCloud, pero no tengo claro cuánto demorará ese paso.

bid-code4dev commented

Gracias por el aviso @danielcarranza , veo que en la linea de consulta generada respondieron con un fix:
Excluding your folder www/lib from the analysis should fix your project, you can set sonar.exclusions to www/lib/**/*, see the documentation here.

Cuando tengan el badge favor de incluirlo en el readme.md con esto podriamos pasar a publicar la herramienta en code.iadb.org. Gracias por todo el trabajo. Un abrazo!

danielcarranza commented

@knlopenknowledge Gracias, lo habíamos visto si :)
Ahora sí quedó andando Quality Gate Status
¡Ya quedó el badge en el readme!

  • 🚀1