Restructure files #56
Conversation
There was a problem hiding this comment.
This is a workaround to keep using pyproject.toml.
The only down side is that now (locally) you'll need to install the fastapi_app package before installing the backend package as the backend package can't resolve the fastapi_app source, I tried specifying the path but it needs a full path which will be a nightmare on different platforms.
https://stackoverflow.com/questions/75159453/specifying-local-relative-dependency-in-pyproject-toml
There was a problem hiding this comment.
Btw we can use a packages manager (poetry or smth) and we will be able to specify a relative path but I know you are not a big fan of this approach so, I didn't do it. But it's an option too.
| ```bash | ||
| python3 -m pip install -e src/backend | ||
| python3 -m pip install -e src | ||
| python ./src/fastapi_app/setup_postgres_database.py | ||
| python ./src/fastapi_app/setup_postgres_seeddata.py | ||
| python ./src/backend/fastapi_app/setup_postgres_database.py | ||
| python ./src/backend/fastapi_app/setup_postgres_seeddata.py |
There was a problem hiding this comment.
This now how it works but it's not pretty due to the issue I mentioned below.
vscode resolution will still work as is and running the launch.json scripts will work as is.
Purpose
Fix #52
Does this introduce a breaking change?
When developers merge from main and run the server, azd up, or azd deploy, will this produce an error?
If you're not sure, try it out on an old environment.
Type of change
Code quality checklist
See CONTRIBUTING.md for more details.
python -m pytest).python -m pytest --covto verify 100% coverage of added linespython -m mypyto check for type errorsruffmanually on my code.