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

Only run tests in Travis for stuff that changed #143

Merged
merged 1 commit into from Jul 12, 2016

Conversation

jeffmo
Copy link
Contributor

@jeffmo jeffmo commented Jun 14, 2016

Adds a flow-typed run-ci-tests command that looks at the diff between the current working directory and origin/master, extracts a list of libdefs that were changed/added, and only runs applicable tests (rather than all tests).

Fixes #76

@jeffmo jeffmo force-pushed the only_run_changed_libdef_tests branch 4 times, most recently from 8a4187c to 9059ca8 Compare June 14, 2016 23:18
@jeffmo
Copy link
Contributor Author

jeffmo commented Jun 16, 2016

(FYI: I'm just waiting on some time to open up so I can debug the travis tests before this can land)

@jeffmo jeffmo force-pushed the only_run_changed_libdef_tests branch 7 times, most recently from 3056b3e to fe2b85a Compare July 11, 2016 22:58
@jeffmo jeffmo force-pushed the only_run_changed_libdef_tests branch from fe2b85a to e3efc23 Compare July 11, 2016 23:03
@jeffmo
Copy link
Contributor Author

jeffmo commented Jul 11, 2016

Woo! Travis build seems to be passing now. I think this is ready for review if anyone feels like running through it before I merge?

(cc @ryyppy @marudor )

@ryyppy
Copy link
Contributor

ryyppy commented Jul 12, 2016

Looking good to me :-)

@marudor
Copy link
Collaborator

marudor commented Jul 12, 2016

I'm all for merging :)

@jeffmo jeffmo merged commit d6399c3 into master Jul 12, 2016
@jeffmo jeffmo deleted the only_run_changed_libdef_tests branch July 12, 2016 14:09
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

3 participants