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

fix: Poetry shell failed due to incorrect pyproject.toml format #1051

Merged
merged 3 commits into from
Dec 17, 2024

Conversation

korjsh
Copy link
Contributor

@korjsh korjsh commented Dec 13, 2024

Problem

The poetry shell command was not working due to missing or incorrect configuration in the pyproject.toml file. Users encountered the following error:

Solution

  • Added the [tool.poetry] section to the pyproject.toml file with the necessary configuration to enable the poetry shell command to work correctly.
  • Configured essential fields such as name, version, and dependencies to meet Poetry's requirements.

Changes Made

  • Updated pyproject.toml:
    • Added [tool.poetry] section.
    • Included required fields (name, version, etc.).
    • Verified the dependencies were correctly defined.

Testing

  • Ran the poetry shell command to ensure a virtual environment is correctly activated without errors.

pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
pyproject.toml Outdated Show resolved Hide resolved
@korjsh korjsh force-pushed the main branch 3 times, most recently from 9449784 to a7ba6b1 Compare December 16, 2024 00:50
pyproject.toml Outdated Show resolved Hide resolved
@vkehfdl1 vkehfdl1 enabled auto-merge (squash) December 16, 2024 11:18
auto-merge was automatically disabled December 16, 2024 23:06

Head branch was pushed to by a user without write access

@korjsh
Copy link
Contributor Author

korjsh commented Dec 16, 2024

@vkehfdl1 Hello, Could you please check if the workflow labeled '1 workflow awaiting approval' is stuck or not being approved?
image

@vkehfdl1 vkehfdl1 enabled auto-merge (squash) December 17, 2024 12:23
@vkehfdl1 vkehfdl1 merged commit 499f54a into Marker-Inc-Korea:main Dec 17, 2024
1 check passed
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