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

Travis builds #32

Merged
merged 23 commits into from Jan 29, 2017

Conversation

Projects
None yet
3 participants
@henryiii
Copy link
Member

henryiii commented Jan 27, 2017

This PR addresses three of the outstanding issues for GooFit 2.0.0, and includes a few bugfixes for the new CMake build. The main features are:

  • Travis builds, with simple example test, closes #31
  • Improved documentation, preparing for (but not including) automatically building the gh-pages branch #28
  • One of the datafiles has been moved to GitHub from Dropbox, auto download #29
  • Change log, closes #27

Several of the issues still need a few more features to be complete, but this PR should be ready.

@hassec

hassec approved these changes Jan 29, 2017

Copy link
Member

hassec left a comment

Just checked out this build and ran some tests.

Noticed two warnings in one of the examples. Not sure why they hadn't surfaced in earlier versions.. But they are fixed now.

Think we can go ahead and merge this.

@henryiii henryiii merged commit 97e131d into master Jan 29, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details

@henryiii henryiii modified the milestone: V 2.0 Jun 8, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment