Skip to content
This repository has been archived by the owner on Apr 27, 2024. It is now read-only.

Feature/hexagonal #16

Merged
merged 7 commits into from
Oct 21, 2023
Merged

Feature/hexagonal #16

merged 7 commits into from
Oct 21, 2023

Conversation

vitor-lupinetti
Copy link
Collaborator

No description provided.

Copy link
Collaborator

@gabriellima09 gabriellima09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Para organização de pastas dos testes, recomendaria termos a seguinte formatação:

src/
test/

No momento, os testes estão dentro* da mesma pasta que o do projeto. Podemos segmentar essa parte para ficar mais perto do padrão mostrado.

Segue o repo de exemplo que vimos na aula de arquitetura hexagonal: https://github.com/rcarneironet/valhalla-hexagonal-architecture/tree/master/test/Valhalla.Tests

Copy link
Collaborator

@gabriellima09 gabriellima09 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Dúvida também para todos

Faria sentido segregar parte do código do projeto de Driven/TechLanches.Infrastructure e ter um projeto de SqlServerAdapter? Assim teríamos a vantagem se ter os repositórios como portas

Copy link
Collaborator

@gabriellima09 gabriellima09 Oct 18, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Mesmo após as mudanças, os projetos ainda tem pastas físicas que ficaram sobrando:

image

image

*Poderia realizar a limpeza e organização dessas pastas?

Copy link
Collaborator Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pra esse caso, acredito que possa ser algum cache, aqui pra mim elas não aparecem mais, mas foi preciso excluir elas na mão mesmo dps do primeiro build com as novas pastas

@gabriellima09
Copy link
Collaborator

No quesito de portas, ainda não definimos nenhuma pasta. Faria sentido mover arquivos para ficar no padrão das portas da aplicação?

@vitor-lupinetti vitor-lupinetti merged commit 5bcef5f into develop Oct 21, 2023
2 checks passed
@vitor-lupinetti vitor-lupinetti deleted the feature/hexagonal branch October 21, 2023 01:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants