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

Use make test to run the tests on travis-ci #78

Merged
merged 2 commits into from Nov 20, 2012
Merged

Conversation

cdeil
Copy link
Contributor

@cdeil cdeil commented Nov 20, 2012

No description provided.

#- echo "LD_LIBRARY_PATH = $LD_LIBRARY_PATH"

# Install rootpy
- time python setup.py install --user
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

We don't need to install anymore, because make test will build in-place and execute all tests.
Was that your question?

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oops, I think we do need it to build the docs.
Putting that line back in.

@pwaller
Copy link
Member

pwaller commented Nov 20, 2012

Heads up, #76 broke this. Please merge or rebase against master and resolve.

cdeil added a commit that referenced this pull request Nov 20, 2012
Use `make test` to run the tests on travis-ci
@cdeil cdeil merged commit aa0cf89 into rootpy:master Nov 20, 2012
@pwaller pwaller mentioned this pull request Dec 11, 2012
@pwaller
Copy link
Member

pwaller commented Dec 11, 2012

xref whatsnew #139

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.

None yet

2 participants