プロダクションコードにconsole.logは含めるかどうか
Closed this issue · 3 comments
mitsuruog commented
FBメッセで少し話題にあった件
console.log系を含めた場合、eslintにて警告が出るようになってます。
開発時はデバック目的で利用したいケースもあり現状のままとしたいですが、プロダクション時は意図せぬ情報流出のおそれがあるので、消したいと思います。
方法としては、console, alert, debuggerを自動で消してくれる
sindresorhus/gulp-strip-debug https://github.com/sindresorhus/gulp-strip-debug を使いたいと思います。
変更点は以下の通り
- jscsタスクを
jscs:devとjscs:buildに分割 jscs:buildでjscs()する前に、gulp-strip-debugモジュールでlogなどを削除
どう思います？
mitsuruog commented
jscsタスクsrcを変更しないので無理でしたね。babel:buildの中でやりたいと思います。
- プロダクション時は警告を出した上、削除する
です。どうでしょう？