search

beatfreaker / text-meme

Generate text meme
MIT License
9 stars 1 forks source link

Add travis badge #5

Closed SamVerschueren closed 8 years ago

SamVerschueren commented 8 years ago

Noticed text-meme and text-meme-cli are not added yet to travis although a .travis.yml file is present.

beatfreaker commented 8 years ago

@SamVerschueren after adding to travis it is failing while installing canvas because of this dependency .... any idea how to resolve it?

SamVerschueren commented 8 years ago

Here you go: https://docs.travis-ci.com/user/installing-dependencies/

beatfreaker commented 8 years ago

Thanks @SamVerschueren configured it but getting some issue in Node.js:stable -> The command "eval npm install " failed. while it works fine with 0.10 and 0.12

any suggestions?

SamVerschueren commented 8 years ago

For the interested people: https://travis-ci.org/beatfreaker/text-meme/jobs/98076679

Not quite sure why this is going wrong.

Have you tried installing text-meme on your own system in Node 5.2.0? Maybe something is wrong with canvas for that version.

beatfreaker commented 8 years ago

@SamVerschueren Yeah Node 5.2.0 was the issue canvas is not working in that environment and now added enabled travis in both module. Thanks for the help :)

SamVerschueren commented 8 years ago

You didn't add the badge to the README yet :).

I noticed you removed the stable entry in .travis.yml. If it's working on Node 4.x, maybe you should add Node 4 as an entry to travis and limit the engines property so that it cannot be installed on Node versions that aren't supported.