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

Troubleshooting docs 244 #225

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

ShayNehmad-RecoLabs
Copy link

@ShayNehmad-RecoLabs ShayNehmad-RecoLabs commented Apr 2, 2021

Resolves #224

image

Signed-off-by: Shay Nehmad <[email protected]>
@nat-n
Copy link
Collaborator

nat-n commented Apr 5, 2021

Hi @ShayNehmad-RecoLabs, thanks for working on this.

Now that I see it explained like this, I'm curious if you tried doing poetry uninstall betterproto; poetry install betterproto, before just rebuilding the env? I guess you did try running poetry install again after updating the prproject.toml?

I'm also still unsure exactly what caused the issue you experienced. It's surprising given that AFAIK neither betterproto nor poetry are doing anything at all unusual with how the entry_point console_script is created/installed into the venv.

I just want to be sure that whatever goes into a troubleshooting section like this represents the problem well and gives the simplest solution.

Also poetry now has a smoother workflow for finding/deleting envs: https://python-poetry.org/docs/managing-environments/#listing-the-environments-associated-with-the-project

@nat-n nat-n added the documentation Improvements or additions to documentation label Apr 6, 2021
@ShayNehmad-RecoLabs
Copy link
Author

I can't really take the time to reproduce - this worked for me, so 🤷

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

Successfully merging this pull request may close these issues.

Enhanchement: Add "troubleshooting" section to the documentation
2 participants