tox/docs/config.rst - temp_dir possible change #2930
Locked
cesarcoatl
started this conversation in
General
Replies: 1 comment 1 reply
-
|
Sounds right! Thanks for spotting. I am happy to review / merge your merge proposal. |
Beta Was this translation helpful? Give feedback.
1 reply
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
Hello,
In one of my projects where I ran
toxI got an error from the IDE I was using saying that.tmpwas not valid, and it turned out that it was referring to.tox/.tmp.When I wanted to change the default name for
temp_dirviatox.iniI noticed a potential mistake in the docs.I've forked this repo and this is the
difffor the change I might propose in a PR if it turns out I'm right.$ git diff diff --git a/docs/config.rst b/docs/config.rst index adc7863..3804acd 100644 --- a/docs/config.rst +++ b/docs/config.rst @@ -174,7 +174,7 @@ Core .. conf:: :keys: temp_dir - :default: {tox_root}/.tmp + :default: {work_dir}/.tmp Directory where to put tox temporary files. For example: we create a hard link (if possible, otherwise new copy) in this directory for the project package. This ensures tox works correctly when having parallel runs (as each sessionWhile...
So as my
diffshows, I believe the docs should be modified.Beta Was this translation helpful? Give feedback.
All reactions