-
Notifications
You must be signed in to change notification settings - Fork 13
Configurar integração contínua CI e Análise estática de código #80
Comments
… de código no README.md Signed-off-by: Eliseu Egewarth <[email protected]>
Devemos definir quais métricas serão coletadas pelo codeClimate afim de definir as métricas que serão analisadas e mantidas no projeto. |
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Métrica:
|
Ao selecionar minhas preferências, levei em conta o nível de conhecimento atual da tecnologias. Por ser um estágio inicial, talvez aplicar as boas práticas de técnicas de programação pode não ser tão fácil. |
Acredito que devemos colocar para a próxima sprint a folha de estilo. Isso vai ajudar a organizar, na comunicação e o aproveitamento de código. |
Ao dar meu feedback levei em consideração que não temos um conhecimento da linguagem, mas levei em conta os comentarios do @LucasAmoedo sobre replicação de código, por isso não aprovei a analise de blocos de código similares. |
Meio de intruso: Se vocês vão aplicar folha de estilo, permitam-me recomendar a da Airbnb para React Native. É possível usar o eslint para testar automaticamente (inclusive fazendo o Travis falhar se algo não estiver de acordo). Como ponto negativo, ela pode ser bastante chata. Bastante. E um pouco difícil. Mas ajuda a padronizar código. |
@LucasAmoedo Obrigada pela dica @LucasAmoedo 👍 😊 👏 |
… de código no README.md Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Das métricas de código listadas e votadas acima foram definidas:
|
… de código no README.md Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
… de código no README.md Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
… de código no README.md Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
…rramenta] Signed-off-by: Eliseu Egewarth <[email protected]>
Signed-off-by: Eliseu Egewarth <[email protected]>
Nessa issue devera ser feito:
The text was updated successfully, but these errors were encountered: