search

anykeyh / clear

Advanced ORM between postgreSQL and Crystal
https://github.com/anykeyh/clear
MIT License
272 stars 34 forks source link

Set up CI/CD pipeline to test the library #204

Open dukenguyenxyz opened 3 years ago

dukenguyenxyz commented 3 years ago

Clear used to have a CI/CD pipeline. I don't know for what reason this was removed. But one issue that was present even when we had that pipeline was that sometimes the specs would pass locally but fail on the pipeline - due to environmental differences.

Whatever the case, I think it is necessary we set up the CI/CD pipeline, I've written docker-compose files to wrap around crystal projects with PostgreSQL using clear multiple times before. This PR solves part of the issue https://github.com/anykeyh/clear/pull/201 , so if you are happy with this idea, @anykeyh , please reply so I can set it up for clear. We can use either travis or github actions, I have used both before, personally I think github action is a better choice.

anykeyh commented 3 years ago

Sorry for late reply. I'm not having a minute for myself until probably March.

I don't understand this issue, there is a travis CI here. Isn't it enough ?