Skip to content

Allow package to be manually specified in get_pkg_data_* functions - #4079

Merged
embray merged 3 commits into
astropy:masterfrom
embray:utils/pkg-data-manual-pkg
Aug 28, 2015
Merged

embray merged 3 commits into
astropy:masterfrom
embray:utils/pkg-data-manual-pkg

Conversation

@embray

@embray embray commented Aug 14, 2015

Copy link
Copy Markdown
Member

Add optional package argument to get_pkg_data_* functions so that package data can be retrieved from packages other than the one the function was called from. I got annoyed in writing some tests that I couldn't do this already.

Also converted most of the examples for those functions into working doctests, which revealed some bugs in both the tests and the functions themselves.

One such test is skipped due to
astropy/astropy-data#4

@embray

embray commented Aug 14, 2015

Copy link
Copy Markdown
Member Author

Huh. I didn't get any of these test failures locally....

@embray

embray commented Aug 14, 2015

Copy link
Copy Markdown
Member Author

Oh wait, that would be why--I only ran the tests with --package=utils.

@embray
embray force-pushed the utils/pkg-data-manual-pkg branch from a61c1e5 to 4cbeca1 Compare August 17, 2015 21:22
@embray

embray commented Aug 17, 2015

Copy link
Copy Markdown
Member Author

Rebased on top of #4083, on which this PR is now a dependency.

@embray
embray force-pushed the utils/pkg-data-manual-pkg branch from 4cbeca1 to e84de5e Compare August 17, 2015 21:48
@astrofrog

Copy link
Copy Markdown
Member

👍

@embray

embray commented Aug 28, 2015

Copy link
Copy Markdown
Member Author

Gonna go ahead and merge this in a sec.

embray added a commit to embray/astropy that referenced this pull request Aug 28, 2015
@embray embray added this to the v1.1.0 milestone Aug 28, 2015
embray added 3 commits August 28, 2015 13:36
package data can be retrieved from packages other than the one the
function was called from.

Also converted most of the examples for those functions into working
doctests, which revealed some bugs in both the tests and the functions
themseves.

One such test is skipped due to
astropy/astropy-data#4
@embray
embray force-pushed the utils/pkg-data-manual-pkg branch from 9a6bf62 to 58a8f9d Compare August 28, 2015 17:38
embray added a commit that referenced this pull request Aug 28, 2015
Allow package to be manually specified in get_pkg_data_* functions
@embray
embray merged commit 77ac03d into astropy:master Aug 28, 2015
@embray
embray deleted the utils/pkg-data-manual-pkg branch August 28, 2015 17:41
dhomeier pushed a commit to dhomeier/astropy that referenced this pull request Dec 17, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants