@flowfuse/driver-docker 2.24.5 → 2.26.0

package/.github/workflows/release-publish.yml

@@ -13,6 +13,6 @@ jobs:
               with:
                   node-version: 18
             - run: npm ci --omit dev
-            - uses: JS-DevTools/npm-publish@7f8fe47b3bea1be0c3aec2b717c5ec1f3e03410b # v4.1.1
+            - uses: JS-DevTools/npm-publish@d2fef917d9aa6e1f0ee5eac28ed023eb4921ce51 # v4.1.3
               with:
                   token: ${{ secrets.NPM_PUBLISH_TOKEN }}

package/CHANGELOG.md

@@ -1,3 +1,13 @@
+#### 2.26.0: Release
+
+ - Bump JS-DevTools/npm-publish from 4.1.2 to 4.1.3 (#176)
+ - Bump JS-DevTools/npm-publish from 4.1.1 to 4.1.2 (#174) @app/dependabot
+ - fix eqeqeq & quotes (#175) @hardillb
+ - Convert to eslint v9 (#173) @hardillb
+
+#### 2.25.0: Release
+
+
 #### 2.24.5: Release
 
  - Update Node.js version in build workflow (#168)

package/docker.js

@@ -357,7 +357,7 @@ module.exports = {
                             container = undefined
                         }
                     } catch (err) {
-                        console.log(err)
+                        console.error(err)
                     }
                     if (container) {
                         const state = await container.inspect()
@@ -420,7 +420,7 @@ module.exports = {
                                     createMQttTopicAgent(broker)
                                 }
                             } catch (err) {
-                                console.log(err)
+                                console.error(err)
                             }
                         }
                     }
@@ -749,7 +749,7 @@ module.exports = {
             await container.stop()
             await container.remove()
         } catch (err) {
-            console.log(err)
+            console.error(err)
         }
     },
     getBrokerAgentState: async (broker) => {

package/eslint.config.mjs

@@ -0,0 +1,56 @@
+import globals from 'globals'
+import js from '@eslint/js'
+import neostandard, { resolveIgnoresFromGitignore } from 'neostandard'
+import stylistic from '@stylistic/eslint-plugin'
+import noOnlyTests from 'eslint-plugin-no-only-tests'
+
+export default [
+    {
+        files: ['**/*.js'],
+        languageOptions: {
+            globals: {
+                ...globals.browser
+            },
+            sourceType: 'script'
+        }
+    },
+    {
+        files: ['test/**/*.js'],
+        languageOptions: {
+            globals: {
+                ...globals.browser,
+                ...globals.mocha
+            },
+            sourceType: 'script'
+        }
+    },
+    {
+        ignores: [
+            ...resolveIgnoresFromGitignore()
+        ]
+    },
+    js.configs.recommended,
+    ...neostandard(),
+    {
+        plugins: {
+            '@stylistic': stylistic,
+            'no-only-tests': noOnlyTests
+        },
+        rules: {
+            // built-in
+            eqeqeq: 'error',
+            'object-shorthand': ['error'],
+            'no-console': ['error', { allow: ['debug', 'info', 'warn', 'error'] }],
+            quotes: ['off', 'error', 'single', { avoidEscape: true }],
+
+            // plugin:stylistic
+            '@stylistic/indent': ['warn', 4], // https://eslint.style/rules/indent#options
+            '@stylistic/spaced-comment': ['error', 'always'], // https://eslint.style/rules/spaced-comment
+            '@stylistic/no-multi-spaces': 'error', // https://eslint.style/rules/no-multi-spaces#no-multi-spaces
+            '@stylistic/comma-dangle': ['error', 'never'], // https://eslint.style/rules/comma-dangle#comma-dangle
+
+            // plugin:no-only-tests
+            'no-only-tests/no-only-tests': 'error'
+        }
+    }
+]

package/package.json

@@ -1,12 +1,12 @@
 {
     "name": "@flowfuse/driver-docker",
-    "version": "2.24.5",
+    "version": "2.26.0",
     "description": "Docker driver for FlowFuse",
     "main": "docker.js",
     "scripts": {
         "test": "echo \"Error: no test specified\" && exit 1",
-        "lint": "eslint -c .eslintrc \"*.js\"",
-        "lint:fix": "eslint -c .eslintrc \"*.js\" --fix"
+        "lint": "eslint -c eslint.config.mjs",
+        "lint:fix": "eslint -c eslint.config.mjs --fix"
     },
     "repository": {
         "type": "git",
@@ -26,7 +26,10 @@
         "got": "^11.8.0"
     },
     "devDependencies": {
-        "eslint": "^8.25.0",
-        "eslint-config-standard": "^17.0.0"
+        "@eslint/js": "^9.39.2",
+        "eslint": "^9.39.2",
+        "eslint-plugin-no-only-tests": "^3.3.0",
+        "globals": "^16.5.0",
+        "neostandard": "^0.12.2"
     }
 }

package/.eslintrc

@@ -1,19 +0,0 @@
-{
-    "root": true,
-    "env": {
-        "browser": true,
-        "commonjs": true,
-        "es2021": true,
-        "mocha": true
-    },
-    "extends": [
-        "standard"
-    ],
-    "parserOptions": {
-        "ecmaVersion": 12
-    },
-    "rules": {
-        "indent": ["error", 4],
-        "object-shorthand": ["error"]
-    }
-}