Skip to content

nose dependency - #29

Merged
astrofrog merged 1 commit into
astropy:masterfrom
mdboom:pywcs
Oct 21, 2011
Merged

nose dependency#29
astrofrog merged 1 commit into
astropy:masterfrom
mdboom:pywcs

Conversation

@mdboom

@mdboom mdboom commented Oct 21, 2011

Copy link
Copy Markdown
Contributor

Hello,

I got this error while running tests:

========================================= ERRORS =========================================
___________________ ERROR collecting astropy/wcs/tests/test_wcsprm.py ____________________
astropy/wcs/tests/test_wcsprm.py:6: in <module>
>   from nose.tools import raises
E   ImportError: No module named nose.tools
========================== 851 passed, 1 error in 4.73 seconds ===========================

Should nose be added as a dependency?

Thanks,
Prasanth

@mdboom

mdboom commented Oct 21, 2011

Copy link
Copy Markdown
Contributor

Thanks for catching this. All the rest of us probably have nose installed so didn't see it. Nose should not a be a dependency -- we are using py.text instead. I will rewrite these tests to not require nose. (Expect a pull request shortly).

@astrofrog

astrofrog commented Oct 21, 2011

Copy link
Copy Markdown
Member

Looks good, thanks!

astrofrog added a commit that referenced this pull request Oct 21, 2011
@astrofrog
Merge pull request #29 from mdboom/pywcs
714ca2c 
nose dependency
@astrofrog
astrofrog merged commit 714ca2c into astropy:master Oct 21, 2011
keflavich pushed a commit to keflavich/astropy that referenced this pull request Oct 9, 2013
@mdboom
Don't import nose -- we're using py.test exclusively. Add a "raises" …
9fb642a 
…decorator to the test helpers. Closes astropy#29.
keflavich pushed a commit to keflavich/astropy that referenced this pull request Oct 9, 2013
@astrofrog
Merge pull request astropy#29 from mdboom/pywcs
e223bc2 
nose dependency
@mdboom
mdboom deleted the pywcs branch May 21, 2014 23:56
embray pushed a commit to embray/astropy that referenced this pull request Jan 23, 2015
@astrofrog
Merge pull request astropy#29 from cdeil/issue_2566
0767e06 
Use -j option for parallel testing
@mdboom mdboom mentioned this pull request Apr 17, 2015
Closed
Cadair added a commit to Cadair/astropy that referenced this pull request Nov 25, 2022
@Cadair
Merge pull request astropy#29 from Cadair/fitsio_compression_tests
5510518 
Fitsio compression tests
jeffjennings pushed a commit to jeffjennings/astropy that referenced this pull request Jul 2, 2025
@eteq
Merge pull request astropy#29 from eteq/add-APE-description
699cba5 
Add paragraph describing APEs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@mdboom @astrofrog