tbrand / neph

A modern command line job processor, similar with make command
https://tbrand.github.io/neph/
MIT License
204 stars 7 forks source link

CI configuration #67

Closed waghanza closed 6 years ago

waghanza commented 6 years ago

Hi,

I can see that travis is using osx to run test phase

The default workers is linux, ubuntu LTS

Why osx was selected ?

Regards,

notramo commented 6 years ago

@tbrand can configure the CI. By the way, why is CI used? It is very slow. Even my 1.8GHz dual core machine (from 2006) is faster. It is a good thing for projects that has wide platform support, and lot of contributors. But Crystal only supports 4-5 architectures currently (only Linux and macOS on different CPUs), and Neph don't has any platform-specific C binding. And only @waghanza and I are the contributors currently, so every change can be tested manually (in less time). Also, it is very annoying when a build fails only because timeout on slow CPUs in CI machines, although the code is fully correct. Could testing be removed and only used for releasing macOS builds?

waghanza commented 6 years ago

@notramo I think CI is a value added to this project, since it use to communicate on stability of this project. For example, a green badge on README, is helpful to grow. As I understand, the main issue you notice is speed / crystal support.

Some CI tools are faster that travis :

but on all crystal support is not official

To ensure speed we can change CI tool, but I think travis is more than enough (since very simple)

My initial question was : why default was changed (perhaps there is a great justification :stuck_out_tongue: )

PS : If CI for releasing is suggest https://cirrus-ci.org, it the only CI I know using

waghanza commented 6 years ago

Related to #68