search

Emplea-do / emplea_do

Open source tech jobs portal. Made with .Net Core
https://emplea.do
MIT License
77 stars 55 forks source link

Agregar un linter para el proyecto para evitar set de cambios sucios #201

Open erickconcepcion opened 4 years ago

erickconcepcion commented 4 years ago

Algunos IDE's pueden ensuciar nuestro código y causar falsos positivos en nuestro set de cambios

Según la observación del pull request #191 al analizar el commit podemos ver que el set de cambios correcto inicia en la lines 297 y termina en la 316, luego todos los demás set de cambios de ese archivos so exactamente el mismo código con la excepción de algunos espacios de menos o espacios de mas posiblemente detonados por el IDE utilizado. image

Descripción de la solucion

Lo ideal seria tener un linter como StyleCop que nos advierta cuando tanto el IDE como nosotros mismos ensuciemos el proyecto y tan pronto vayamos a realizar un commit procedamos a limpiar.

Alternativas a considerar

StyleCop, SonarLint, Roslyn

erickconcepcion commented 4 years ago

Voy a usar StyleCop porque se agrega en el proyecto y no necesita que quien este desarrollando instale nada adicional en su VS y por lo que se podrá ver los resultados en el build output.