Conversation
|
I have some use cases that require rye run to preserve the current working directory, maybe this could be configurable in that case? |
|
@bluss I see three approaches:
|
|
I'm not sure what's wrong with the current behaviour. Here's an argument in favour of keeping it. These two should be equivalent: They will only be the same if they both behave the same w.r.t cwd. |
|
Current behavior makes relative file path in the cmd does not work, see #930 |
|
Note that this function also handles rye run python, rye run cmd where cmd is an executable in the venv too (take for example cmd = pytest). Maybe it makes sense for tool.rye.scripts by itself, yes! |
|
I feel like making it run from root is a reasonable default as many scripts will be taking relative file arguments. However I do think that if we change that, then an option should be added to override it like PDM does now. Also I do like what npm does with |
Fixes #930