forked from RasaHQ/rasa
-
Notifications
You must be signed in to change notification settings - Fork 0
Sprint 5 Clean Code e SOLID
Iuri Severo edited this page Feb 21, 2021
·
2 revisions
- 17/11/2020 a 26/11/2020
- Alexandre Miguel
- Letícia Meneses
- Lucas Dutra
- Sara Silva
- Henrique Martins
- Rômulo Souza
- Martha Dantas
Atividade | Descrição |
---|---|
Issue | #3153 - consider replacing os.path uses with pathlib |
Pull Request | #7383 |
O que foi feito | Foi alterado no cógido a forma que são manipulados os paths e diretórios do sistema (lib os para lib pathlib), para maior legibilidade. Devido ao grande número de alterações que a issue demanda, o mantenedor sugeriu que fossem feitos vários PRs para a issue. Nessa sprint a dupla trabalhou nos módulos core e utils . |
Problemas | O PR anterior ainda não foi merjado e por isso não conseguimos aproveitar algumas melhorias que já fizemos |
Solução | - |
Atividade | Descrição |
---|---|
Issue | Não criada ainda |
Pull Request | Não criada ainda |
O que foi feito | Refatoração (Clean Code) da classe Domain
|
Problemas | - |
Solução | - |
Descrição detalhada | Refatoração - Domain |
Atividade | Descrição |
---|---|
Issue | Não criada ainda |
Pull Request | Não criada ainda |
O que foi feito | Refatoração (SOLID) da classe Validator
|
Problemas | - |
Solução | - |
Descrição detalhada | Refatoração - Validator |
Atividade | Descrição |
---|---|
Issue | Não criada ainda |
Pull Request | Não criada ainda |
O que foi feito | Refatoração (Clean Code) do método test_count_vector_featurizer_persist_load
|
Problemas | Trecho de código repetitivo e com declaração ruim |
Solução | Desacoplamento da lógica de atribuição para uma função |
Descrição detalhada | Refatoração - test_count_vector_featurizer_persist_load |
Atividade | Descrição |
---|---|
Issue | Não criada ainda |
Pull Request | #12 |
O que foi feito | Refatoração (Clean Code) do método test_mitie_featurizer_train
|
Problemas | A função tinha problemas com Complexidade Cognitiva e número de linhas além do permitido. |
Solução | Refatoração para usar um método implementado que diminuia tais problemas sem gerar erros. |
Descrição detalhada | Refatoração - test_mitie_featurizer_train |
Atividade | Descrição |
---|---|
Issue | Modificações associadas à Issue #7022 - Check if conversation ID is valid for all server endpoints |
Pull Request | Modificações associadas ao PR #7238 Validate Conversation ID |
O que foi feito | Aplicação de técnicas de Clean Code nos módulos de server e tracker_store
|
Problemas | - |
Solução | - |
Descrição detalhada | Refatoração (Clean Code) do arquivo server.py nas funções ensure_conversation_exists e no arquivo tracker_store.py
|
Atividade | Descrição |
---|---|
Issue | #2811 - Split long modules / Improve directory structure |
Pull Request | - |
O que foi feito | Nenhuma atividade foi realizada nesse sprint. |
Problemas | - |
Solução | - |