remove astropy/data directory #466

Merged
merged 2 commits into from Nov 14, 2012

Projects

None yet

4 participants

@eteq
Owner
eteq commented Nov 7, 2012

This just removes the astropy/data directory.

Someone can correct me if I'm mis-remembering, but I believe this was originally intended for all data, but now that the data directories live in packages, this directory is not necessary. It may need to be resurrected if something in the astropy root needs data, but until then we may as well get rid of it.

Member
embray commented Nov 7, 2012

Okay.

Owner

+1

Owner

Although I'm not sure I understand why Travis is failing...

Member
mdboom commented Nov 8, 2012

The test_local_data_name test is trying to open the README.rst that used to exist in the data directory. I think that test can probably be updated to load any other random file.

Owner
eteq commented Nov 9, 2012

Good point @mdboom - I'll hold off on this until #425 is done then, as it will require modifying a test that will be moved by that PR

eteq added some commits Nov 7, 2012
@eteq eteq remove astropy/data directory
It was originally intended for all data, but now that the data
directories live in packages, it is not currently necessary
a62ba25
@eteq eteq commented-out test using root data dir 20095a3
Owner
eteq commented Nov 13, 2012

Alright, rebased and updated to fix the failing test. Assuming travis passes, I'll go ahead and merge this.

@eteq eteq merged commit f465e53 into astropy:master Nov 14, 2012

1 check passed

default The Travis build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment