Browse Source

Merge pull request #9029 from Snuffleupagus/eslint--report-unused-disable-directives

Enable the `--report-unused-disable-directives` ESLint command line option
Tim van der Meij 8 years ago committed by GitHub
parent
commit
815bc53a16
  1. 3
      gulpfile.js
  2. 1
      src/doc_helper.js
  3. 1
      src/pdf.worker.entry.js

3
gulpfile.js

@ -1160,7 +1160,8 @@ gulp.task('lint', function (done) {
console.log('### Linting JS files'); console.log('### Linting JS files');
// Ensure that we lint the Firefox specific *.jsm files too. // Ensure that we lint the Firefox specific *.jsm files too.
var options = ['node_modules/eslint/bin/eslint', '--ext', '.js,.jsm', '.']; var options = ['node_modules/eslint/bin/eslint', '--ext', '.js,.jsm', '.',
'--report-unused-disable-directives'];
var esLintProcess = spawn('node', options, { stdio: 'inherit', }); var esLintProcess = spawn('node', options, { stdio: 'inherit', });
esLintProcess.on('close', function (code) { esLintProcess.on('close', function (code) {
if (code !== 0) { if (code !== 0) {

1
src/doc_helper.js

@ -12,7 +12,6 @@
* See the License for the specific language governing permissions and * See the License for the specific language governing permissions and
* limitations under the License. * limitations under the License.
*/ */
/* eslint-disable strict */
/* /*
NOTE: This file is created as a helper to assist with JSDoc html files. NOTE: This file is created as a helper to assist with JSDoc html files.

1
src/pdf.worker.entry.js

@ -12,7 +12,6 @@
* See the License for the specific language governing permissions and * See the License for the specific language governing permissions and
* limitations under the License. * limitations under the License.
*/ */
/* eslint-disable strict */
(typeof window !== 'undefined' ? window : {}).pdfjsDistBuildPdfWorker = (typeof window !== 'undefined' ? window : {}).pdfjsDistBuildPdfWorker =
require('./pdf.worker.js'); require('./pdf.worker.js');

Loading…
Cancel
Save