Fix Firstrade package metadata#109
Conversation
There was a problem hiding this comment.
💡 Codex Review
Here are some automated review suggestions for this pull request.
Reviewed commit: 07181d0cff
ℹ️ About Codex in GitHub
Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you
- Open a pull request for review
- Mark a draft as ready
- Comment "@codex review".
If Codex has suggestions, it will comment; otherwise it will react with 👍.
Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".
| [tool.setuptools] | ||
| py-modules = [ | ||
| "decision_mapper", | ||
| "main", |
There was a problem hiding this comment.
Add Flask to the packaged app dependencies
With this commit main becomes part of the installable distribution, but main.py imports from flask import ... at module import time while the pyproject.toml dependencies still do not install Flask. In a fresh environment that follows the README’s pip install -e . flow or installs the wheel and then imports main/runs gunicorn main:app, the app fails immediately with ModuleNotFoundError, so the package metadata needs to declare Flask when exposing this entrypoint.
Useful? React with 👍 / 👎.
Summary
Tests