search

editorconfig / editorconfig-core-java

Clone of EditorConfig core written in Java
Apache License 2.0
61 stars 26 forks source link

Activate Travis CI hook #4

Closed ppalaga closed 9 years ago

ppalaga commented 9 years ago

Travis CI descriptor to run the core tests was added in 10c8d1f. The Travis CI hook needs to be activated by some of the repo owners in https://github.com/editorconfig/editorconfig-core-java/settings/hooks

xuhdev commented 9 years ago

Done

xuhdev commented 9 years ago

Would you mind also adding OpenJDK as a testing case? Some people (e.g. myself) prefer an open source Java implementation. Thanks!

ppalaga commented 9 years ago

@xuhdev you mean in .travis.yml? Having both Oracle and OpenJDK there makes no sense from technical point of view because they are identical in all important respects for a library like this. Having both would just double Travis build times. Switching to OpenJDK is OK, I prepared a PR that you may want to review and eventually merge : https://github.com/editorconfig/editorconfig-core-java/pull/5