Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/201 Agregación de un linter para el proyecto #212

Open
wants to merge 1 commit into
base: development
Choose a base branch
from

Conversation

erickconcepcion
Copy link
Contributor

@erickconcepcion erickconcepcion commented Jul 31, 2020

What's new?

  • Se instaló el paquete StyleCop.Analyzers en todo el proyecto.
  • Se suprimieron reglas modificaban mucho el código.
  • Se corrigió cada una de las violaciones de las reglas que se permitieron.

Lint Rules

  • Opening braces should not be preceded by blank line.
  • Code should not contain trailing whitespace.
  • Not allow blank line after close braces.
  • Code should not contain multiple whitespace characters in a row.
  • Code should not contain multiple blank lines in a row.
  • Single line comment should begin with a space.
  • Code should not contain blank lines at the end of the file.
  • Use built-in type alias.
  • Closing parenthesis should be on line of last parameter.
  • Opening parenthesis should not be preceded by a space.

Additional considerations

En caso de que aparezca otra advertencia que no sea ningunas de las mencionadas en el punto anterior si es deseable suprimirla debe agregar la regla al final del archivo emplea_do\StyleCop.ruleset en el siguiente formato:
<Rule Id="SAXXXX" Action="None" /> <!-- Descripción de la advertencia -->
Donde el Id es el código de la advertencia que desea suprimir y en un comentario su description. Esto es con el propósito de que en un futuro se decida agregar a nuestro set de reglas.

Issue: #201

- Se intalo el paquete en todo el proyecto.
- Se suprimieron reglas modificaban mucho el codigo.
- Se corrigio cada una de las violacionesde las reglas que se permitieron.
@eatskolnikov
Copy link
Collaborator

Do a rebase

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants