-
Notifications
You must be signed in to change notification settings - Fork 633
Use the Python 3 super() syntax #597
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
Conversation
Codecov Report
@@ Coverage Diff @@
## develop #597 +/- ##
==========================================
Coverage ? 64.18%
==========================================
Files ? 63
Lines ? 5721
Branches ? 0
==========================================
Hits ? 3672
Misses ? 2049
Partials ? 0 |
Codecov Report
@@ Coverage Diff @@
## develop #597 +/- ##
===========================================
+ Coverage 63.98% 63.99% +0.01%
===========================================
Files 63 63
Lines 5573 5572 -1
===========================================
Hits 3566 3566
+ Misses 2007 2006 -1 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Super
@hardbyte Appveyor has problems and does not execute thests any more. See the last commit on this PR: it does not have the Appveyor entry in the test/CI list. This could be a clue: https://ci.appveyor.com/project/hardbyte/python-can/builds/24650559 |
It seems to be working again here: https://ci.appveyor.com/project/hardbyte/python-can/builds/24651358 |
Updates the code in
can/**
&test/**
.This can only be merged once #594 got merged.