Skip to content
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

Setup/switch to astropy helpers #144

Merged
merged 21 commits into from
May 27, 2014

Conversation

wkerzendorf
Copy link
Member

This is now an update to the new astropy setup helpers that make the update of any installation scripts much easier.

@ssim @orbitfold @mklauser can one of you sign off on this?

@wkerzendorf
Copy link
Member Author

@embray @astrofrog - Hey guys, I've updated this package now to use the new package-template structure. It works really well and updating was a breeze!!! Thanks for that.

There are some minor issues that I still have. Currently running tests seems only possible for me using the following command line ASTROPY_USE_SYSTEM_PYTEST=1 python setup.py test --skip-docs

I think for most packages it would be great if they could use the system pytest if there is one installed. The other problem I ran into is

usage: -c [options] [file_or_dir] [file_or_dir] [...]
-c: error: unrecognized arguments: --doctest-rst

the only way to get rid of it is to use the command line option --skip-docs

If you guys could have a quick look that would be great.

@wkerzendorf
Copy link
Member Author

@embray @astrofrog I fixed the doctest problem - there is now only the question how to enable the system pytest by default.

@ssim
Copy link
Contributor

ssim commented May 27, 2014

Sounds good to me!

ssim added a commit that referenced this pull request May 27, 2014
@ssim ssim merged commit 5e7b790 into tardis-sn:master May 27, 2014
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.

2 participants