ispyb / EXI

Extended User Interface for ISPyB based on ISPyB's API
4 stars 14 forks source link

Fix the warnings issues when launching grunt, because too many lead to error #238

Open delageniere opened 7 years ago

delageniere commented 7 years ago

Take into account the result of grunt to fix the small warnings in js files, because if too many warnings it gives an error

dsclassen commented 6 years ago

I'll second this. After 578 warning grunt just bailed out. I had to use the --force flag to get grunt to complete.

js/saxs/widget/specimenwidget.js
  line 87    col 33   Wrapping non-IIFE function literals in parens is unnecessary.
  line 110   col 30   'specimen' is already defined.
  line 115   col 39   'onSuccess' is already defined.
  line 115   col 41   Wrapping non-IIFE function literals in parens is unnecessary.
  line 131   col 26   'specimen' is already defined.
  line 139   col 2    Unnecessary semicolon.

  ⚠  578 warnings

Warning: Task "jshint:prod" failed. Use --force to continue.

Aborted due to warnings.