-
-
Notifications
You must be signed in to change notification settings - Fork 18.4k
TST: Period tests reorg (gh14854) #15352
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
whoosh that |
hahah yeah. As a result, |
yep but let's wait on that for now |
@TrigonaMinima going to merge shortly. |
Codecov Report@@ Coverage Diff @@
## master #15352 +/- ##
==========================================
+ Coverage 86.32% 86.32% +<.01%
==========================================
Files 141 141
Lines 51163 51163
==========================================
+ Hits 44166 44167 +1
+ Misses 6997 6996 -1
Continue to review full report at Codecov.
|
thanks again @TrigonaMinima ! if you are up for more :> let's move the remainder of tseries/tests to pandas/tests/tseries also let's remove tests/timeseries_legacy.py (and some of data can be removed as well). BUT should move any useful tests to a proper location. IOW the pickle things you can simply remove (they are really old). but some of the freq tests might be useful. |
Sure. I'll try to do it tonight. I should also thank you. You are helping me alot throughout the way. Is there something remaining to be done in this PR? |
closed via 9ba1008 @TrigonaMinima no this is all good. thanks! |
git diff upstream/master | flake8 --diff
Period
tests.