質問をすることでしか得られない、回答やアドバイスがある。

15分調べてもわからないことは、質問しよう!

ただいまの
回答率

90.62%

  • Node.js

    1792questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • gulp

    265questions

    gulpは、Node.jsをベースとしたタスク自動化ツールの一つ。ストリームでファイルを処理することが特徴です。CSSプリプロセッサの使用時のコンパイルや、CSS・JavaScriptファイルの圧縮・結合などを自動的に行うことができます。

Gulp JSHintでError Warningsが無い時だけ次のタスクを実行してリロードさせたい

解決済

回答 1

投稿 編集

  • 評価
  • クリップ 0
  • VIEW 1,103

KiKiKi_KiKi

score 563

gulp.watchでjavascriptのファイルを監視。変更があった時下記のような処理を行わせたいです。

1. JSHintでチェック
2. 問題がなければconcatなどで結合
3. 全てのタスクが終了したらbrowser-syncで画面をリロード

grupfile.jsを作成したのですが、下記の問題が解消できておりません。
a. JSHintで問題が検出されても次のタスクを実行してしまう
b. JSHintで問題があると1度はbrowser-syncでリドードできるが、
   その後javascriptファイルを変更してもタスクが実行されなくなってしまう。
   (javascriptだけwatchされなくなる?)

  "devDependencies": {
    "browser-sync": "^2.9.1",
    "gulp": "^3.9.0",
    "gulp-concat": "^2.6.0",
    "gulp-if": "^1.2.5",
    "gulp-jshint": "^1.11.2",
    "gulp-load-plugins": "^0.10.0",
    "jshint-stylish": "^2.0.1",
    "main-bower-files": "^2.9.0",
    "run-sequence": "^1.1.2"
  }

gulpfile.js
var gulp        = require("gulp"),
    $           = require('gulp-load-plugins')(),
    browserSync = require('browser-sync'),
    runSequence = require('run-sequence'),
    concat      = require("gulp-concat"),
    jshint      = require("gulp-jshint"),
    reload      = browserSync.reload;

gulp.task('jshint', function () {
  return gulp.src(devDir + '/js/*.js')
    .pipe( jshint() )
    .pipe( jshint.reporter('jshint-stylish') )
    .pipe( $.if(!browserSync.active, jshint.reporter('fail')) );
    // .pipe( reload({stream: true, once: true}) );
    // ↑ここにオプションを与えてリロードする処理を書けば
    // JSHintでエラーがあっても watch されなくなることは無いが
    // エラーの有無に関わらずリロードされてしまう
});

gulp.task('concat.js', function(cb) {
  return gulp.src(devDir + '/js/*.js')
    .pipe( concat('main.js') )
    .pipe( gulp.dest( destDir + '/js' ) );
});

// Javascript Task
gulp.task('js', function(cb) {
  runSequence('jshint', 'concat.js', cb);
  // runSequence('jshint', 'concat.js', reload);
  // ↑ 最後にreloadのタスクを実行させるようにすると
  //  JSHintでエラーなどがあると、その後 gulp.task('js')が実行されなくなる
  //  リロードを別の gulp.task() を作って実行しようとしても同様

  // runSequence('jshint', 'concat.js', reload, cb);
  // ↑ エラーになり実行できない書き方
});

// watch
gulp.task('default', function() {
  gulp.watch([devDir + '/js/*.js'], ['js']);
});

タスクの最後にオプションを与えてリロードの処理を実行させるようにすれば、watchされなくなる問題は発生しないようなので、個人的にはJSHintで問題が検出されたら次のタスクを実行しないようにできれば、問題は解決できそうかなと思っています。
この問題の解決に至る事ができなかったので質問させて頂きました。
何卒よろしくお願いします。

[参考]
Web Starter Kit で JavaScript のライブラリを利用しようとしたらエラーで gulp のサーバーが止まってしまう場合の対応 - HAM MEDIA MEMO
  • 気になる質問をクリップする

    クリップした質問は、後からいつでもマイページで確認できます。

    またクリップした質問に回答があった際、通知やメールを受け取ることができます。

    クリップを取り消します

  • 良い質問の評価を上げる

    以下のような質問は評価を上げましょう

    • 質問内容が明確
    • 自分も答えを知りたい
    • 質問者以外のユーザにも役立つ

    評価が高い質問は、TOPページの「注目」タブのフィードに表示されやすくなります。

    質問の評価を上げたことを取り消します

  • 評価を下げられる数の上限に達しました

    評価を下げることができません

    • 1日5回まで評価を下げられます
    • 1日に1ユーザに対して2回まで評価を下げられます

    質問の評価を下げる

    teratailでは下記のような質問を「具体的に困っていることがない質問」、「サイトポリシーに違反する質問」と定義し、推奨していません。

    • プログラミングに関係のない質問
    • やってほしいことだけを記載した丸投げの質問
    • 問題・課題が含まれていない質問
    • 意図的に内容が抹消された質問
    • 広告と受け取られるような投稿

    評価が下がると、TOPページの「アクティブ」「注目」タブのフィードに表示されにくくなります。

    質問の評価を下げたことを取り消します

    この機能は開放されていません

    評価を下げる条件を満たしてません

    評価を下げる理由を選択してください

    詳細な説明はこちら

    上記に当てはまらず、質問内容が明確になっていない質問には「情報の追加・修正依頼」機能からコメントをしてください。

    質問の評価を下げる機能の利用条件

    この機能を利用するためには、以下の事項を行う必要があります。

回答 1

check解決した方法

0

JSHintでエラーが発生した時にタスクをそこで止めて、かつwatchが切れないようにする方法にて自己解決しました。
使用しているプラグインのバージョンは質問欄を参照ください。

どうやらjshint.reporter('fail')でstreamが停止するという問題は修正されていたっぽく
.pipe( $.if(!browserSync.active, jshint.reporter('fail')) ) とせず
単にpipe( jshint.reporter('fail') )でそれ以降のタスクを実行しないようにできました。

また、JSHintでエラー/ワーニングがあるとjavascriptファイルだけ監視されなくなる問題も
最後のタスクに.pipe( reload({stream: true, once: true}) );を付けて実行するか
reloadというタスクを作成しrunSequence('jshint', 'taskName', ... 'reload', cb)とすることでJSHintで問題があってもファイルの監視が切れる事は無くなりました。

 EX: gulpfile.js

var gulp        = require("gulp"),
    $           = require('gulp-load-plugins')(),
    browserSync = require('browser-sync'),
    runSequence = require('run-sequence'),
    concat      = require("gulp-concat"),
    jshint      = require("gulp-jshint");

gulp.task('jshint', function () {
  return gulp.src(devDir + '/js/*.js')
    .pipe( jshint() )
    .pipe( jshint.reporter('jshint-stylish') )
    .pipe( jshint.reporter('fail') ); // ← 変更
});

gulp.task('concat.js', function(cb) {
  return gulp.src(devDir + '/js/*.js')
    .pipe( concat('main.js') )
    .pipe( gulp.dest( destDir + '/js' ) );
});

// Reload TASK
gulp.task('reload', function(cb) {
  return gulp.src('/')
    .pipe( browserSync.reload({stream: true, once: true}) );
});

// Javascript Task
gulp.task('js', function(cb) {
  runSequence('jshint', 'concat.js', 'reload', cb);
});

// watch
gulp.task('default', function() {
  gulp.watch([devDir + '/js/*.js'], ['js']);
});


jshint.reporter('fail') when used in an async task doesn't fail the build #28
Web Starter Kit で JavaScript のライブラリを利用しようとしたらエラーで gulp のサーバーが止まってしまう場合の対応 - HAM MEDIA MEMO

ただ、gitHubのチケット上の解決したコメントは1 Jul 2014で、問題があったってblogの日時は2014-07-05なので、どうなんだろう?その後さらにアップデートされて問題が解決したのでしょうか???

ちょっとリロードタスクの実効のさせ方がキモチワルイですが、一旦はコレで意図した挙動になったので解決とさせていただきます。

投稿

  • 回答の評価を上げる

    以下のような回答は評価を上げましょう

    • 正しい回答
    • わかりやすい回答
    • ためになる回答

    評価が高い回答ほどページの上位に表示されます。

  • 回答の評価を下げる

    下記のような回答は推奨されていません。

    • 間違っている回答
    • 質問の回答になっていない投稿
    • スパムや攻撃的な表現を用いた投稿

    評価を下げる際はその理由を明確に伝え、適切な回答に修正してもらいましょう。

15分調べてもわからないことは、teratailで質問しよう!

  • ただいまの回答率 90.62%
  • 質問をまとめることで、思考を整理して素早く解決
  • テンプレート機能で、簡単に質問をまとめられる

関連した質問

同じタグがついた質問を見る

  • Node.js

    1792questions

    Node.jsとはGoogleのV8 JavaScriptエンジンを使用しているサーバーサイドのイベント駆動型プログラムです。

  • gulp

    265questions

    gulpは、Node.jsをベースとしたタスク自動化ツールの一つ。ストリームでファイルを処理することが特徴です。CSSプリプロセッサの使用時のコンパイルや、CSS・JavaScriptファイルの圧縮・結合などを自動的に行うことができます。