Skip to content

ewoks install: poetry ignores an active virtual or conda environment - #346

Merged
woutdenolf merged 1 commit into
mainfrom
fix_poetry_active_environment
Oct 8, 2026
Merged

woutdenolf merged 1 commit into
mainfrom
fix_poetry_active_environment

Conversation

@woutdenolf

Copy link
Copy Markdown
Member

PR summary

ewoks install with poetry installed the workflow requirements in the active environment (poetry run, an activated venv or conda env) instead of the new one.

  • Poetry commands no longer inherit VIRTUAL_ENV and CONDA_PREFIX.
  • Added a test for both cases.

AI Disclosure

  • No AI used
  • AI tool Claude detected the problem when testing different package managers in the same shell

Co-Authored-By: Claude Opus 5.5 (1M context) <noreply@anthropic.com>
@woutdenolf
woutdenolf requested a review from a team October 8, 2026 12:14
@codecov

codecov Bot commented Oct 8, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@LudoBroche LudoBroche left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Tested on my machine, and the bug is fixed.
Everything looks good to me.

Maybe add a changelog entry if relevant

@woutdenolf

Copy link
Copy Markdown
Member Author

Since poetry support is not released yet we don't need the fix in the changelog imo.

@woutdenolf
woutdenolf merged commit f623d2d into main Oct 8, 2026
12 checks passed
@woutdenolf
woutdenolf deleted the fix_poetry_active_environment branch October 8, 2026 12:54
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