-
Notifications
You must be signed in to change notification settings - Fork 33
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
six usage is redundant #390
Comments
Hi @pgajdos ! Yes, that's a good suggestion. This will also require a bit of code cleanup as well... Do you think you can take a look at that and get rid of it? I'll take a look otherwise... |
Nazdar! This works with 'not (integration or test_get_port_mappings or selinux or ownership)' pytest run, detailed build log can be found here: I am not 100% sure this is complete, though. Petr |
Nazdar :) I could apply your patch, but it'd be better if you created a pull request with the change so that
|
I think it is clear I am unsure I can do an upstream class submission, but here you are. Sure we can discuss, but I do not aspire to be author of this change if not necessary ;). |
https://build.opensuse.org/request/show/1073107 by user mcepl + dimstar_suse - do not require six - added patches fix user-cont/conu#390 + python-conu-no-six.patch
As you declare that this package is 3.6+, could you please consider drop of dependency on
six
please?The text was updated successfully, but these errors were encountered: