search

project-hatohol / hatohol

A unified manager of monitoring software
http://www.hatohol.org/
Other
89 stars 29 forks source link

Change jshint to eslint #2559

Closed otakuto closed 7 years ago

otakuto commented 7 years ago

es6対応のためeslintに切り替えました.

masa0612 commented 7 years ago

対応ありがとうございます。 Travisの結果を見るに，チェックは正常に動いているように見えます。 しかしwarningだけでもexit codeが1となり，その時点でテストが終了しています。

「warningを全て修正するか」「warningではexit codeは1にはならない」のどちらかが必要な対処となりますが， 前者はwarningの数が2000弱であまり現実的ではないので，後者の修正をお願いできませんか？

masa0612 commented 7 years ago

これテストコケてるのってこのPRのせいじゃない。このPR自体問題なさそうだから，aコミット削除して再度PRお願いします。

cosmo0920 commented 7 years ago

:tada: