Add Alembic Migration environment to the project, patch model architecture

python-discord / api

A FastAPI-based service that allows our services to communicatie with our database.

MIT License

10 stars 1 forks source link

Closed D0rs4n closed 2 years ago

D0rs4n commented 3 years ago

Closes #7 (Finally 😁)

This PR also adds the migration steps to the Dockerfile.
Introduces patches in the Database models, (mostly patches relationship - cascading problems)

D0rs4n commented 3 years ago

Pushed a new commit, everything should be in order now.

D0rs4n commented 3 years ago

Looks good. Just one suggestion to add black (only checking) to the lint test workflow.

That is definitely worth considering, but unfortunately it is out of the scope of this PR. Thank you for the thorough review! :)