Skip to content

ewoks install: print execute and remove commands that work from any terminal - #362

Merged
woutdenolf merged 2 commits into
mainfrom
install_summary_commands
Nov 3, 2026
Merged

woutdenolf merged 2 commits into
mainfrom
install_summary_commands

Conversation

@woutdenolf

@woutdenolf woutdenolf commented Oct 24, 2026 •

Copy link
Copy Markdown
Member

PR summary

Chain of PRs for #341, install the environment of a workflow with only a package manager. Merge in order: #361 → #362 → #363 → #348 → #364

  • The summary of ewoks install prints commands that work from any terminal and directory:
    • Execute: <env python> -m ewoks execute <absolute workflow path> instead of ewoks execute --env <env> <workflow>.
    • Paths are quoted. On Windows the commands use PowerShell syntax (Remove-Item instead of rmdir).
  • With --in-place, the summary now shows the python and the execute command. Before, it showed nothing.

AI Disclosure

  • AI tool Claude used to create the code, tests and docs.

🤖 Generated with Claude Code

…erminal

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
@codecov

codecov Bot commented Oct 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 37.03704% with 17 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/ewoks/__main__.py 37.03% 17 Missing ⚠️

📢 Thoughts on this report? Let us know!

Comment thread src/ewoks/__main__.py Resolved
Comment thread src/ewoks/__main__.py Resolved
@woutdenolf
woutdenolf changed the base branch from requirements_ewoks_version to main November 3, 2026 10:45
@woutdenolf
woutdenolf merged commit 2e8436b into main Nov 3, 2026
13 checks 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