Skip to content

Conversation

arokem
Copy link
Owner

@arokem arokem commented Jan 24, 2015

@dhermes - how does this look to you?

If you think this is ready to go, I will merge it.

Then, we can close #113

@arokem
Copy link
Owner Author

arokem commented Jan 24, 2015

Hmm. Apparently not: https://travis-ci.org/arokem/python-matlab-bridge/builds/48195292

Any ideas?

@dhermes
Copy link
Contributor

dhermes commented Jan 24, 2015

I suppose we need to add a do nothing script. My bad.

How about something like this (added to .travis.yml):

script:
  - true

If you have unit tests you'd like to run you could add a section for that as well.

@arokem
Copy link
Owner Author

arokem commented Jan 24, 2015

That would require matlab...

On Sat, Jan 24, 2015 at 2:50 PM, Danny Hermes notifications@github.com
wrote:

I suppose we need to add a do nothing script. My bad.

How about something like this (added to .travis.yml):

script:

  • true

If you have unit tests you'd like to run you could add a section for that
as well.


Reply to this email directly or view it on GitHub
#114 (comment)
.

@dhermes
Copy link
Contributor

dhermes commented Jan 24, 2015

Haha yes I thought about this a bit. Doesn't seem to be worth the effort.

@isbadawi
Copy link
Collaborator

You could maybe run the tests under Octave (set USE_OCTAVE=1 environment
variable)?
On Jan 24, 2015 5:55 PM, "Danny Hermes" notifications@github.com wrote:

Haha yes I thought about this a bit. Doesn't seem to be worth the effort.


Reply to this email directly or view it on GitHub
#114 (comment)
.

@isbadawi
Copy link
Collaborator

It would probably be trickier than that actually. I'll look into myself
later.
On Jan 24, 2015 6:42 PM, "Ismail Badawi" ismail@badawi.io wrote:

You could maybe run the tests under Octave (set USE_OCTAVE=1 environment
variable)?
On Jan 24, 2015 5:55 PM, "Danny Hermes" notifications@github.com wrote:

Haha yes I thought about this a bit. Doesn't seem to be worth the effort.


Reply to this email directly or view it on GitHub
#114 (comment)
.

@arokem
Copy link
Owner Author

arokem commented Jan 25, 2015

I'll merge this one for now. I think that we should put out a new release/tag soon anyway, so we will get to try this out!

arokem added a commit that referenced this pull request Jan 25, 2015
@arokem arokem merged commit d375ff8 into master Jan 25, 2015
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.

3 participants