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

$POE_ROOT/$POE_PWD unavailable in tool.poe.env section #198

Closed
tumidi opened this issue Feb 22, 2024 · 3 comments
Closed

$POE_ROOT/$POE_PWD unavailable in tool.poe.env section #198

tumidi opened this issue Feb 22, 2024 · 3 comments
Labels
bug Something isn't working

Comments

@tumidi
Copy link

tumidi commented Feb 22, 2024

I'd expect $POE_ROOT/$POE_PWD variables to get expanded in [tool.poe.env].

[tool.poe.env]
GLOBAL_POE_ROOT = "${POE_ROOT}"  # doesn't work

[tool.poe.tasks.my-task]
default_item_type = "cmd"
sequence = [
  "echo GLOBAL_POE_ROOT=${GLOBAL_POE_ROOT}",
  "echo TASK_POE_ROOT=${TASK_POE_ROOT}",
]

[tool.poe.tasks.my-task.env]
TASK_POE_ROOT = "${POE_ROOT}"  # works

I created a minimal reproduction: https://github.com/tumidi/poethepoet-env-test

Output is:

$ poe my-task
GLOBAL_POE_ROOT=
TASK_POE_ROOT=/tmp/poethepoet-env-test
@nat-n
Copy link
Owner

nat-n commented Feb 22, 2024

Hi @tumidi, yes I would expect that too. Thanks for reporting this.

@nat-n nat-n added the bug Something isn't working label Feb 22, 2024
@nat-n
Copy link
Owner

nat-n commented Apr 27, 2024

This was incidentally fixed by the recent refactor of config management that is now on the development branch.

nat-n added a commit that referenced this issue Apr 27, 2024
nat-n added a commit that referenced this issue Apr 27, 2024
nat-n added a commit that referenced this issue Apr 27, 2024
nat-n added a commit that referenced this issue Apr 27, 2024
nat-n added a commit that referenced this issue Apr 27, 2024
@nat-n
Copy link
Owner

nat-n commented Apr 27, 2024

This issue is now fixed with 0.26.0 🚀

@nat-n nat-n closed this as completed Apr 27, 2024
tumidi added a commit to questionpy-org/qpy-dev that referenced this issue May 13, 2024
The new version fixes a bug that has been affecting us.

Refs: nat-n/poethepoet#198
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants