Merge 739f3c8 into 1aeac3b

.eslintrc.js

@@ -1,18 +1,21 @@
 "use strict";
 
 module.exports = {
+  root: true,
+  reportUnusedDisableDirectives: true,
   env: {
     node: 1,
     mocha: 1,
   },
   parserOptions: {
     ecmaVersion: 2021,
   },
-  plugins: ["node", "eslint-plugin", "prettier"],
+  plugins: ["node", "eslint-plugin"],
   extends: [
     "eslint:recommended",
     "plugin:node/recommended",
     "plugin:eslint-plugin/all",
+    "prettier",
   ],
   rules: {
     "arrow-body-style": "error",
@@ -31,6 +34,5 @@ module.exports = {
           "https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/{{name}}.md",
       },
     ],
-    "prettier/prettier": "error",
   },
 };

.github/workflows/node.js.yml → .github/workflows/ci.yml

@@ -1,4 +1,4 @@
-name: Node.js CI
+name: CI
 on:
   push:
     branches: ["main"]
@@ -10,21 +10,22 @@ jobs:
     strategy:
       matrix:
         os: [ubuntu-latest, windows-latest]
-        node-version: [18.16.0, 20.2.0]
+        node-version: [18.18.2, 20.9.0]
     steps:
       - name: Set git to use LF
         run: |
           git config --global core.autocrlf false
           git config --global core.eol lf
-      - uses: actions/checkout@v2
+      - uses: actions/checkout@v4
       - name: Use Node.js ${{ matrix.node-version }}
-        uses: actions/setup-node@v2
+        uses: actions/setup-node@v4
         with:
           node-version: ${{ matrix.node-version }}
           cache: "npm"
       - run: npm ci
+      - run: npm run format:check
+      - run: npm run lint
       - run: npm test
-      - run: npm run coverage
       - name: Coveralls
         uses: coverallsapp/github-action@master
         with:

.tool-versions

@@ -1 +1 @@
-nodejs 20.2.0
+nodejs 20.9.0

README.md

@@ -50,10 +50,10 @@ Or add the "recommended" preset:
 
 |     |     | Name                                                                                                                      | Description                                                                |
 | --- | --- | ------------------------------------------------------------------------------------------------------------------------- | -------------------------------------------------------------------------- |
-| ✔   | 🔧  | [no-absolute-imports](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/no-absolute-imports.md) | disallow absolute imports for files that are within the current package    |
+| ✔  | 🔧  | [no-absolute-imports](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/no-absolute-imports.md) | disallow absolute imports for files that are within the current package    |
 |     |     | [no-cross-imports](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/no-cross-imports.md)       | disallow imports of files that are inside another package                  |
-| ✔   | 🔧  | [no-relative-imports](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/no-relative-imports.md) | disallow relative imports of files that are outside of the current package |
-| ✔   |     | [require-dependency](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/require-dependency.md)   | disallow importing from packages that are not listed as a dependency       |
+| ✔  | 🔧  | [no-relative-imports](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/no-relative-imports.md) | disallow relative imports of files that are outside of the current package |
+| ✔  |     | [require-dependency](https://github.com/joshuajaco/eslint-plugin-workspaces/blob/main/docs/rules/require-dependency.md)   | disallow importing from packages that are not listed as a dependency       |
 
 ## Presets
 

lib/rules/no-absolute-imports.js

@@ -36,8 +36,8 @@ module.exports.create = (context) => {
             fixer.replaceTextRange(
               [start + 1, end - 1],
               pathToImport(
-                relative(dirname(filename), path.replace(name, location))
-              )
+                relative(dirname(filename), path.replace(name, location)),
+              ),
             ),
         });
       }

lib/rules/no-cross-imports.js

@@ -71,7 +71,7 @@ module.exports.create = (context) => {
   if (!workspaces) return {};
 
   const forbidden = workspaces.filter(
-    ({ package: { name } }) => !allowed.includes(name)
+    ({ package: { name } }) => !allowed.includes(name),
   );
 
   return getImport(
@@ -83,8 +83,8 @@ module.exports.create = (context) => {
           filterSharedPackagesInCurrentScope(
             currentWorkspace,
             scopedEnabled,
-            scopedSharingFolderName
-          )
+            scopedSharingFolderName,
+          ),
         )
         .forEach(({ package: { name }, location }) => {
           if (
@@ -98,6 +98,6 @@ module.exports.create = (context) => {
             });
           }
         });
-    }
+    },
   );
 };

lib/rules/no-relative-imports.js

@@ -34,11 +34,11 @@ module.exports.create = (context) => {
             fix: (fixer) =>
               fixer.replaceTextRange(
                 [start + 1, end - 1],
-                path.replace(location, name)
+                path.replace(location, name),
               ),
           });
         }
       });
-    }
+    },
   );
 };

lib/rules/require-dependency.js

@@ -44,6 +44,6 @@ module.exports.create = (context) => {
           });
         }
       });
-    }
+    },
   );
 };

lib/utils.js

@@ -42,7 +42,7 @@ const getImport = (workspaces, filename, callback) => {
   if (!workspaces) return {};
 
   const currentWorkspace = workspaces.find(({ location }) =>
-    isSubPath(location, filename)
+    isSubPath(location, filename),
   );
 
   if (!currentWorkspace) return {};
@@ -59,7 +59,7 @@ const getImport = (workspaces, filename, callback) => {
           (node.callee.type === "Identifier" && node.callee.name === "require"))
       ) {
         callback(
-          resolveImport(filename, node, node.arguments[0], currentWorkspace)
+          resolveImport(filename, node, node.arguments[0], currentWorkspace),
         );
       }
     },