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

Code entrypoint #169

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

Code entrypoint #169

wants to merge 2 commits into from

Conversation

jaredoconnell
Copy link
Member

Type of change

  • Refactor
  • New feature
  • Bug fix
  • Optimization
  • Documentation Update

Description

This simply adds a way to run the backend within an IDE debugger in case you want to use a debugger.

I wanted to offer the code to everyone in case others found this valuable.

Checklist before requesting a review

  • I have performed a self-review of my code.

Testing

This is not something that needs tests. I used this my entire time developing for the CPT dashboard without an issue.


The file `app/entrypoint.py` starts the backend using uvicorn on port 8000.

To use a debugger, run the entrypoint file from the debugger with the appropriate poetry backend.
Copy link
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This sounds potentially interesting; but can you elaborate on "appropriate poetry backend"??

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How do you think this should be worded?
Basically it needs Poetry's Venv with the appropriate dependencies included. In pycharm it shows up as "Poetry" in the list of python backends.

@jaredoconnell
Copy link
Member Author

I'm not sure what the error is. It doesn't look related to any changes, unless somehow the use of the dependency is causing problems.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants