Closed RemyDelion closed 6 years ago
Thank you very much for your PR. It looks good to me but it needs one more review.
@stratakis could you please take a look?
I am not so familiar with python-ethtool's codebase to do a review of such a new feature. @hroncok would you be able to chip in?
Also I see that some executable bits were removed in this PR (permissions changing from 755 to 644) on some files. Is this intentional?
Concerning the permissions, no, it wasn't intentional. I didn't want to change any permission.
@hroncok would you be able to chip in?
Not really, sorry.
Okay, our last chance, @encukou?
I just fixup the file permissions (to 755).
That sounds great to me :)
Okay, I don't see any reason why not merge this PR.
Thank you for your contribution!
Thanks!
https://github.com/fedora-python/python-ethtool/issues/37