-
Notifications
You must be signed in to change notification settings - Fork 34
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
ValueError: need more than 1 value to unpack #104
Comments
Just noticed tox moved to 3.0.0 |
I would not rule it out. Can you pin tox to 2.9.1 and check of the error persists? Thanks! |
I may have found the culprit . I had
in https://github.com/asmodehn/rostful/blob/6839b8820e88992257ee9cfe21c63aa16d71882b/tox.ini#L36 After changing it like this, travis |
Thanks. To clarify: it had nothing to with the 3.0 reease then? or did that:
work pre 3.0 and stopped working? |
I reverted to use latest tox, and everything went fine, so it's not related to tox 3.0 I have been playing with my tox configuration recently, so I am ready to bet that
never worked as I thought it would.
|
From where I stand at the moment definitely a bad idea, because the tox.ini parser is already quite involved :) |
I'm also leery of doing that. All three positions (in this case |
Alright then, feel free to close this issue. |
from https://travis-ci.org/asmodehn/rostful/jobs/367548379/config
The text was updated successfully, but these errors were encountered: