jshint / jshint / 2706

100%

push

travis-ci

[[DOCS]] Expand warning message

`export` declarations with empty export export clauses are not
necessarily superfluous: if they also specify a "from" clause, they will
cause the referenced module to be evaluated. Because of this, the
warning produced in these cases should be contextual--it should
recommend removal only if no "from" clause is present. In the presence
of such a clause, the warning should instead suggest use of a
binding-free `import` declaration (as this clearly communicates the
intention to evaluate a module without requesting any bindings).

Notes on compatibility:

- The previous warning behavior had not yet been published in a stable
  release (it was implemented at the same time as the parser was
  corrected to support `export` declarations without bindings). This
  makes this change in warning behavior backward-compatible with prior
  releases.
- This warning is relevant for `import` declarations as well. However,
  because prior releases of JSHint included correct parsing support for
  binding-free `import` declarations, introducing an equivalent warning
  would constitute a backwards-breaking change.

8 of 8 new or added lines in 1 file covered. (100.0%)

4015 of 4066 relevant lines covered (98.75%)

14211.29 hits per line