-
Notifications
You must be signed in to change notification settings - Fork 182
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
Use statements #547
Use statements #547
Conversation
Codecov Report
@@ Coverage Diff @@
## master #547 +/- ##
==========================================
+ Coverage 87.50% 87.53% +0.02%
==========================================
Files 88 88
Lines 5931 5993 +62
==========================================
+ Hits 5190 5246 +56
- Misses 741 747 +6
Continue to review full report at Codecov.
|
dd4d8e9
to
fc474ce
Compare
37ac236
to
e073336
Compare
@@ -1,3 +1,11 @@ | |||
use foo::bar as baz |
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.
the double lines are a bit ugly here, but Im inclined to keep things simple and leave it alone
e073336
to
7b05ac4
Compare
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.
Looks great!
What was wrong?
We originally had Python style import statements.
How was it fixed?
Replaced
import
statements withuse
statements.To-Do