posit-dev / posit-dev/rsconnect-python
Should remove `actions.deploy_python_fastapi`
Open
Nobody has claimed this yet.
- Dominant language
- Python
- Stars
- 37
- Forks
- 28
- Avg merge
- 1d 3h
- Merged PRs (30d)
- 7
Description
The function actions.deploy_python_fastapi is a deprecated code path, and many similar functions were removed in #566. This function was kept because it is used by vetiver here.
At some point, we should:
- Replace
actions.deploy_python_fastapiwith one that calls the python fastapi deployment code inmain.py. - Update vetiver to call the function from
main.py. - After some time, remove
actions.deploy_python_fastapi.
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
Start by locating actions.deploy_python_fastapi and the Python FastAPI deployment code in main.py. Then inspect vetiver/rsconnect.py at the linked call site to understand the external caller. Done means vetiver uses the main.py entry point and the deprecated actions function can be removed after the transition period.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- python
- Domain
- backend, cli
- Issue type
- Refactor
- Difficulty
- 4/5
- Estimated time
- 3-5 days
- Activity status
- Stale
- Clarity
- Mostly clear
- Newbie friendliness
- 35/100