Closed nethsix closed 8 years ago
@nethsix Thanks, can I merge this ?
@Arafatk If everyone's ok with Circle CI, you can first sign up for CircleCI here with a few clicks: https://github.com/integrations/circle-ci
After that, at the dashboard:
That should be it. You can then merge. It almost does nothing for now, because circle.yml config is set to perform CI only for branches that contains the string 'circle_ci'. Thus only the CI will happen for this merged branch (you may have to manually trigger it from the dashboard)
@Arafatk Thanks! I'll do another pull request with a new circle.yml within a day or two, if you have signed/set up Circle CI then when I do the pull request we should be able to see CircleCI perform the build and tests (fingers crossed).
@Arafatk great to see you made it work using Docker!
I have two questions though:
git clone https://github.com/Arafatk/ruby-tensorflow.git
(which should be ../tensorflow.rb
btw.), so it will pull from the master branch instead of the branch of the current PR. @chrhansen