search

matiasdelellis / facerecognition

Nextcloud app that implement a basic facial recognition system.
GNU Affero General Public License v3.0
517 stars 46 forks source link

Continuous integration support #41

Closed stalker314314 closed 5 years ago

stalker314314 commented 5 years ago

Add support for Travis. With existing tests. Code coverage is bonus:)

stalker314314 commented 5 years ago

Heya @matiasdelellis, I am trying to do this, but I am not seeing this project in travis here. Can you somehow go to settings and make sure this repo is set to "ON"? Maybe I will see it after that. I kind of started working on this on this branch, but it is all "theory" for now, as I cannot get to Travis to try it out:) Thanks!

stalker314314 commented 5 years ago

Yap, I think I cannot do it, as I am not owner of this repo, says here. @matiasdelellis, you have to set this repo for continuous integration first (and hopefully, I can see it after that)

stalker314314 commented 5 years ago

If you don't see it soonish, I just figured out I can just fork repo, then I will be owner, so I can try it like this:D

matiasdelellis commented 5 years ago

Ohhh.. I had not understood this well .. :disappointed_relieved:

I already registered in travis, and activate the repository.. Please tell me what is needed..

stalker314314 commented 5 years ago

I am not sure how this works, but maybe Travis is not finding .travis.yml file and that's why I am not seeing this repo. Let me merge my PR for this and then we can see where we are and what else is needed.

stalker314314 commented 5 years ago

OK, @matiasdelellis - merged. Travis is almost passing, but you need to onboard matiasdelellis/facerecognition repo to https://scrutinizer-ci.com, I cannot do it:

pic

Let me know once you do this and we should have first passing build:)

P.S. If there is some options over Scrutinizer to give me some role (contributor, admin...), please add me, it will be faster to iterate that way

matiasdelellis commented 5 years ago

The access privileges for "Branko Kokanovic" were updated. :wink:

stalker314314 commented 5 years ago

Thanks! So, with badges, I think we are all good for this, we got some basic stuff, that's it:) Will maybe beef it up once more important stuff are out of the way