Merge pull request #2692 from chmelevskij/chore/sonar-cloud-eslint-match

chore: add no-vars rule to match sonar cloud

.eslintrc.js

@@ -16,5 +16,6 @@ module.exports = {
         "eol-last": "error",
         semi: "error",
         "comma-dangle": ["error", "always-multiline"],
+        "no-var": "error",
     },
 };

src/js/main.js

@@ -364,7 +364,6 @@ function startProcess() {
                     case 'setup_osd':
                         TABS.setup_osd.initialize(content_ready);
                         break;
-
                     case 'configuration':
                         TABS.configuration.initialize(content_ready);
                         break;

src/js/protocols/stm32usbdfu.js

@@ -12,6 +12,9 @@
 */
 'use strict';
 
+// Task for the brave ones. There are quite a few shadow variables which clash when
+// const or let are used. So need to run thorough tests when chaning `var`
+/* eslint-disable no-var */
 var STM32DFU_protocol = function () {
     this.callback = null;
     this.hex = null;