BotLino / Lino

Lino, a chatbot to University of Brasilia!
https://botlino.github.io/docs/
GNU General Public License v3.0
10 stars 2 forks source link

Configurar Code Climate para monitorar a qualidade de código do Lino #17

Closed guiaugusto closed 5 years ago

guiaugusto commented 5 years ago

Descrição

Para um melhor monitoramento da qualidade de código dos arquivos contidos no Lino, é necessário a configuração de uma ferramenta que seja capaz de dar um feedback mais claro a respeito da qualidade de código do repositório do Lino. Com isso, pode-se encontrar demais problemas a serem documentados e resolvidos, para uma melhor manutenibilidade de código.

Tarefas

Critérios de Aceitação

aamandafelix commented 5 years ago

Em um análise inicial do Code Climate, baseada na branch devel, o projeto foi classificado como F Os principais problemas encontrados foram:

Utilizando-se de alguns plugins do Codeclimate , algumas outras questões foram apontadas, como:

Para mais detalhes: https://codeclimate.com/github/BotLino/Lino https://codeclimate.com/github/BotLino/Lino/pull/34

A proposta é utilizar as seguintes configurações:

De acordo com as prioridades do projeto, é interessante retirar algumas dessas análises citadas ou até acrescentar alguma outra?

Lista de plugins disponíveis: https://docs.codeclimate.com/docs/list-of-engines

guiaugusto commented 5 years ago

Gostei muito de suas propostas para uma melhora na análise do Bot, principalmente do uso dos plugins que auxiliam na melhor visualização da qualidade.

Como você deu uma proposta em relação aos problemas que estamos tendo com o Lino, incrementa muito mais na contribuição sua com o Lino. Apoio a utilização de todas as citadas e não vejo a necessidade de outra, por hora.

Nesta issue mesmo, com o estado dela, a ideia final seria colocar as badges solicitadas acima no README do projeto. Em questão ao que precisa ser melhorado, será criado posteriormente outras issues para aumentar a classificação.

aamandafelix commented 5 years ago

A badge de manutenabilidade foi colocada, mas como não existe suíte de testes atualmente no Lino, não há como avaliar a cobertura através do Code Climate e gerar badge de testes.

guiaugusto commented 5 years ago

Faz sentido... Então a issue está completa... Quando iniciarmos a construção dos testes relacionados ao Lino, incluímos uma task para adicionar no README.