i18njs icon indicating copy to clipboard operation
i18njs copied to clipboard
verified publisher icon roddeh

[Idea] CI with Travis

Open marverix opened this issue 6 years ago • 2 comments

How about integration with Travis? Whenever someone will create PR, Travis would run tests - so you will know if it's safe to merge it or not? :)

marverix avatar Feb 05 '19 20:02 marverix

I am not familiar with Travis. Generally I am open to any improvements to tooling so long as they don't create hard dependencies (I am regretting CoffeeScript for example).

roddeh avatar Feb 06 '19 17:02 roddeh

I'm using CS 1.x in my job (as when I came it was already used). Yes, it has it downsides, but for the time it was ok. I still like it. But if you regret having it in CS how about moving it to ES6, and as bundler using rollup.js + babel plugin (check my repo for example: https://github.com/marverix/rollup-plugin-copier ).

So how about this roadmap:

  1. Move tests to Mocha (#10).
  2. Move CoffeeScript to ES6.
  3. Run tests if we didn't brake anything.
  4. Add Linter (eslint) (#11)
  5. Add Travis (#12).
  6. Add husky (https://github.com/typicode/husky).

Ofc I will be glad to help you with above, especially that I'm using above technologies e.g. in repo that I mentioned above.

Br, Marek

marverix avatar Feb 07 '19 07:02 marverix