Skip to content

Conversation

@bsipocz
Copy link
Member

@bsipocz bsipocz commented Feb 23, 2015

Removing version for unused packages, and adding astropy version.

@coveralls
Copy link

Coverage Status

Coverage remained the same at 66.6% when pulling 46f3996 on bsipocz:test_header_customization into 54e6899 on astropy:master.

@keflavich
Copy link
Contributor

conftest.py is inherited from package-template, so I don't really know what this does. Can you explain it a little more?

@bsipocz
Copy link
Member Author

bsipocz commented Feb 24, 2015

@keflavich - The PYTEST_HEADER_MODULES dictionary was introduced in astropy v1.0 (in astropy/astropy#3157) to allow affiliated packages to add the version number of their extra dependencies to the list in the header prints when test are run.
For astroquery it looks like this:

Numpy: 1.9.1
Scipy: 0.15.0
Matplotlib: 1.5.x
h5py: not available

Some of these are irrelevant as the package is not using them, while e.g. astropy is missing. With this PR:

Numpy: 1.9.1
astropy: 1.1.dev12025

keflavich added a commit that referenced this pull request Feb 24, 2015
Customizing test header version information
@keflavich keflavich merged commit 615fc10 into astropy:master Feb 24, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants