From d0dd2c3a9e0b0b394040f93e3230c67530d8c2c3 Mon Sep 17 00:00:00 2001 From: Sebastian Benz Date: Mon, 20 Mar 2023 19:36:40 +0100 Subject: [PATCH] Eslint: don't allow var (#856) --- .eslintrc.js | 1 + api-samples/printing/printers.js | 2 +- functional-samples/sample.bookmarks/popup.js | 4 +++- 3 files changed, 5 insertions(+), 2 deletions(-) diff --git a/.eslintrc.js b/.eslintrc.js index f5abd1aeda..f52b454f5f 100644 --- a/.eslintrc.js +++ b/.eslintrc.js @@ -4,6 +4,7 @@ module.exports = { plugins: ['prettier'], rules: { 'prettier/prettier': ['error'], + 'no-var': ['error'], 'no-unused-vars': [ 'warn', { diff --git a/api-samples/printing/printers.js b/api-samples/printing/printers.js index 7704aa9e9a..43ec5f6542 100644 --- a/api-samples/printing/printers.js +++ b/api-samples/printing/printers.js @@ -3,7 +3,7 @@ // found in the LICENSE file. function onPrintButtonClicked(printerId, dpi) { - var ticket = { + let ticket = { version: '1.0', print: { color: { type: 'STANDARD_MONOCHROME' }, diff --git a/functional-samples/sample.bookmarks/popup.js b/functional-samples/sample.bookmarks/popup.js index d993af4dc1..86437711bb 100644 --- a/functional-samples/sample.bookmarks/popup.js +++ b/functional-samples/sample.bookmarks/popup.js @@ -29,6 +29,7 @@ function dumpTreeNodes(bookmarkNodes, query) { } function dumpNode(bookmarkNode, query) { + let span = ''; if (bookmarkNode.title) { if (query && !bookmarkNode.children) { if ( @@ -51,7 +52,7 @@ function dumpNode(bookmarkNode, query) { chrome.tabs.create({ url: bookmarkNode.url }); }); - var span = $(''); + span = $(''); const options = bookmarkNode.children ? $('[Add]') : $( @@ -182,6 +183,7 @@ function dumpNode(bookmarkNode, query) { } const li = $(bookmarkNode.title ? '
  • ' : '
    ').append(span); + if (bookmarkNode.children && bookmarkNode.children.length > 0) { li.append(dumpTreeNodes(bookmarkNode.children, query)); }