-
-
Notifications
You must be signed in to change notification settings - Fork 18.6k
PKG: Exclude data test files. #19535
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
Merged
Merged
Changes from 3 commits
Commits
Show all changes
40 commits
Select commit
Hold shift + click to select a range
4d77cd8
PKG: Exclude data test files.
TomAugspurger 270e442
Stuff
TomAugspurger 26e9b4b
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger 1804bcc
Refactor data path handling
TomAugspurger 7022152
More fixtures
TomAugspurger 080f000
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger 151ffda
Updated html
TomAugspurger d9d6570
Remove os.path.joins
TomAugspurger 5849591
More modules
TomAugspurger 31fb0b6
Some more
TomAugspurger 9193f15
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger e897f11
Updated packers
TomAugspurger 9cf30fd
Pickle
TomAugspurger 95cde7a
Linting
TomAugspurger 10ddddc
Autouse stata
TomAugspurger e1ea208
Remove filename
TomAugspurger 8616878
Autouse in merge_asof
TomAugspurger 77bf77c
Cleanup plotting
TomAugspurger 156e14b
CLN: Simplify fspath
TomAugspurger f3f3662
Refactor sql tests
TomAugspurger 2cd9706
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger aac3606
Fixed docstrings
TomAugspurger 762a2d1
Moved
TomAugspurger 7c44b77
Use fixture for iris plotting
TomAugspurger ad09951
Abs path for file test
TomAugspurger 6f02d6b
Removed stdout capture from sql tests
TomAugspurger 489f540
Merge remote-tracking branch 'origin/master' into package-size
TomAugspurger 8b42d1c
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger ee4fefd
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger bac438c
Cleanup Manifest
TomAugspurger 84ccdbf
fixed test test
TomAugspurger c4802db
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger 7fd7660
Fixed windows
TomAugspurger c187f8b
whatsnew
TomAugspurger 632a61d
Clarify note [ci skip]
TomAugspurger b5b70c7
TST: refactored html tests
TomAugspurger 9954bba
Remove auto-generated html fixtures
TomAugspurger c771885
linting
TomAugspurger dd75270
Removed test test file
TomAugspurger dbe0c57
Merge remote-tracking branch 'upstream/master' into package-size
TomAugspurger File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -37,6 +37,11 @@ Documentation Changes | |
- | ||
- | ||
|
||
Build Changes | ||
------------- | ||
|
||
- The source and binary distributions no longer include test files, resulting in smaller download sizes. Tests relying on these files will be skipped when using ``pandas.test()``. (:issue:`19320`) | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. test files -> "test data files" or "data files for testing" |
||
|
||
.. _whatsnew_0232.bug_fixes: | ||
|
||
Bug Fixes | ||
|
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
can you add a ref here as well
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.
General note: IMO it is not needed to always ask this of contributors, as this ref is only needed when we actually want to make an explicit link to it from within the rst files (and chances are quite high we will never do this). The ref can always be added at the moment one adds a link.
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.
sure, but in general its a good practice