-
-
Notifications
You must be signed in to change notification settings - Fork 512
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
detox --version doesn't print detox version #284
Comments
Is this really an enhancement and not a bug? |
point taken :) |
Here we are a year later, and still no version info.
Workaround:
|
@markhu I think you have a different detox there. |
Yeah that's node detox not python. 😀 |
Huh, the (Python) detox's --version is, you could argue, sort of fixed?
|
Hi @mgedmin, detox is a both a a tox plugin, that registers itself with tox and adds an entry point that calls tox in a distributed way (code). It acts as a wrapper around tox, which also means that if you use functionality from tox it will work in the context of tox. So asking for the non existing version functionality in detox, will result in AFAICT detox is pretty abandoned at the moment - at least I don't know of anyone that feels responsible for it, so I think it should really be merged into core, if enough people use it, so that it is a proper part of tox, instead of this odd one out. See #439. |
If detox should stay a plugin I think it should also be turned into a "real" one that as called tox-xdist or something like that, deprecating detox as a standalone tool. That would clear up this confustion also. |
Hi @markhu, if you have the energy to complain about the task priorization in a volunteer-driven project, maybe next time you want to use that energy to check first if you are complaining in the right project? Or even better: don't complain at all and ask how you can help, if it is really important for you? Thanks. |
reopened - because it is still a bug if I ask detox for its version and it gives me the tox version instead. Even if the detox version is also shown. If I have a lot of plugins installed it might drown in the visual noise and it is at the very least confusing for the user. |
@obestwalter if there is a hook for the |
Hi @pawamoy, no nothing like that. It uses the prepare function from tox directly, so asking for the version or help ends up directly in tox: https://github.com/tox-dev/detox/blob/0a9fb58ebbf73015f055c549d0c6d68455c335b3/detox/main.py#L10-L13 which goes to: https://github.com/tox-dev/detox/blob/0a9fb58ebbf73015f055c549d0c6d68455c335b3/detox/main.py#L6-L8 You would have to catch the version arg before and print it out directly. |
Kamino closed and cloned this issue to tox-dev/detox |
It's nice that
detox --version
prints the version of tox, but it should also print its own version.The text was updated successfully, but these errors were encountered: