search

diasurgical / devilution

Diablo devolved - magic behind the 1996 computer game
Other
8.77k stars 920 forks source link

Add building check. #27

Closed alexpantyukhin closed 6 years ago

alexpantyukhin commented 6 years ago

I guess it could be useful to add travis for checking at least compiling of the project.

ghost commented 6 years ago

Refer to #20. You'll have to run it by me, how exactly does Travis work? I've heard about it here and there, does it test the code against multiple compilers or something?

alexpantyukhin commented 6 years ago

Actually I'm not much experienced there :) But I found an example: https://github.com/LibreSprite/LibreSprite/blob/master/.travis.yml Maybe possbile exclude "test" and keep just "make".

Predelnik commented 6 years ago

Travis is pretty simple it just runs specified script on provided VM after each commit/PR and checks that it succeds (returns 0) otherwise it sends you an e-mail or blocks PR from being merged.

And yes it can test several compilers/platforms but for windows compilers you have to use the alternative called AppVeyor.

Saibamen commented 6 years ago

With Travis CI we can run tests, builds with multiple environments (compilers), Linux and OSX and we can do code deployments. I created simple setup: #34

For more, please see Travis config for my repo (PHP): https://travis-ci.org/Saibamen/HotelManager

ghost commented 6 years ago

Closing this as of #34, we can track progress of AppVeyor in #20.