Skip to content
This repository has been archived by the owner on Apr 8, 2019. It is now read-only.

Start using the ci-scripts submodule #42

Merged
merged 3 commits into from Jan 25, 2017
Merged

Conversation

prateek
Copy link
Contributor

@prateek prateek commented Jan 24, 2017

  • Got rid of the old .ci scripts. There were 2 which were unused afaict: test.sh and build-common.sh.
  • Started using the new submodule
  • Some tweaks to the makefile to use the new test-cover.sh file
  • Migrated env var package to travis.yml

JFYI, anyone with an existing repo will have to update their existing submodules by doing the following -

git submodule update --init --recursive 

If that doesn't work you may have to delete the folders .ci, and .git/modules/.ci manually and retry the update.

/cc @robskillington @xichen2020 @kobolog @cw9 @martin-mao @ben-lerner

@coveralls
Copy link

coveralls commented Jan 24, 2017

Coverage Status

Coverage increased (+0.1%) to 84.781% when pulling 8ea8877 on prateek/ci-scripts-refactor into dd72a7c on master.

Copy link
Contributor

@xichen2020 xichen2020 left a comment

Choose a reason for hiding this comment

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

LGTM

@prateek prateek merged commit 9dd1dcc into master Jan 25, 2017
@prateek prateek deleted the prateek/ci-scripts-refactor branch January 25, 2017 16:58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants