-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
[CLN] cy cleanup, de-duplication #21826
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
looks fine. may want to run a perf check as touching some sensitive paths. |
Traveling today, will run asv when home. It should be a clear but very
small improvement.
…On Mon, Jul 9, 2018 at 7:52 AM Jeff Reback ***@***.***> wrote:
looks fine. may want to run a perf check as touching some sensitive paths.
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#21826 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AHtGeKQPUh8dHkqNsKzOa0T71CV1h4AIks5uE1IPgaJpZM4VHkva>
.
|
Codecov Report
@@ Coverage Diff @@
## master #21826 +/- ##
=======================================
Coverage 91.9% 91.9%
=======================================
Files 160 160
Lines 49892 49892
=======================================
Hits 45852 45852
Misses 4040 4040
Continue to review full report at Codecov.
|
asv is raising git-related errors for me on both linux and mac. Is anyone else getting these? |
Results are probably meaningless, but at least they're all good-meaningless.
|
thanks ! |
Small cleanups, remove some unnecessary casting, add typing in timezones.