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

reestructure the entire fucking thing #10

Open
wants to merge 6 commits into
base: main
Choose a base branch
from

Conversation

matheusfiorin
Copy link
Owner

@matheusfiorin matheusfiorin commented Dec 29, 2023

Related to: #7

Rationale:

In order to use the correct hexagonal architecture we must change all of the project structure into the correct patterns that this architectural design proposes. We had some misunderstandings about how it has to be.

This PR will need to:

  • create the right ports part of the structure, with the internal/external ports;
  • separate correctly the adapters;
  • separate correctly the core of the application;

This will allow us to have a more clear and concise application helping us to maintain and understand the flow of the data within our application.

@matheusfiorin matheusfiorin added the chaos call the cops label Dec 29, 2023
@JohnPCarvalho
Copy link
Collaborator

JohnPCarvalho commented Jan 4, 2024

I already merged my changes from the PR #8.

We can continue this task now. Remember: we will need to rebase with main and work with what we have here.

@JohnPCarvalho JohnPCarvalho marked this pull request as ready for review January 5, 2024 02:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chaos call the cops
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants