Skip to content

Add .travis.yml from affiliated package template#21

Merged
RayPlante merged 1 commit into
astropy:masterfrom
cdeil:travis-ci
Oct 9, 2013
Merged

Add .travis.yml from affiliated package template#21
RayPlante merged 1 commit into
astropy:masterfrom
cdeil:travis-ci

Conversation

@cdeil
Copy link
Copy Markdown
Member

@cdeil cdeil commented Aug 13, 2013

I think getting continuous integration running on travis-ci will help ... I'd recommend adding this now and then switching on Python 3 tests, Sphinx docs build tests, ... as these things become available.

@cdeil
Copy link
Copy Markdown
Member Author

cdeil commented Aug 19, 2013

@RayPlante Do you want this? Anything I should change?

@cdeil
Copy link
Copy Markdown
Member Author

cdeil commented Sep 19, 2013

I've switched on Python 3 testing on travis-ci.

Locally I get an error:

$ python3.2 setup.py build
Traceback (most recent call last):
  File "setup.py", line 35, in <module>
    import pyvo
  File "/Users/deil/code/pyvo/pyvo/__init__.py", line 156, in <module>
    from . import registry
  File "/Users/deil/code/pyvo/pyvo/registry/__init__.py", line 8, in <module>
    from .vao import *
  File "/Users/deil/code/pyvo/pyvo/registry/vao.py", line 848
    delim = u"#"
               ^
SyntaxError: invalid syntax

But independently of this error I think you should switch on travis-ci testing ... testing every commit with Python 2 and 3 before merging prevents most regressions.

@cdeil
Copy link
Copy Markdown
Member Author

cdeil commented Sep 19, 2013

travis-ci sees the same Python 3 issue:
https://travis-ci.org/cdeil/pyvo/jobs/11566014#L139

RayPlante added a commit that referenced this pull request Oct 9, 2013
Add .travis.yml from affiliated package template
@RayPlante RayPlante merged commit 56aa86f into astropy:master Oct 9, 2013
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