@arcgis/eslint-config 4.34.0-next.4 → 4.34.0-next.40
This diff represents the content of publicly available package versions that have been released to one of the supported registries. The information contained in this diff is provided for informational purposes only and reflects changes between package versions as they appear in their respective public registries.
- package/dist/config/index.js +3 -3
- package/dist/config/lumina.js +1 -1
- package/dist/{makePlugin-Ca089k86.js → makePlugin-BWc0voYM.js} +1 -1
- package/dist/plugins/lumina/index.js +2 -2
- package/dist/plugins/webgis/index.js +5 -4
- package/package.json +5 -5
- /package/dist/plugins/webgis/rules/{require-js-in-core-import.d.ts → require-js-in-imports.d.ts} +0 -0
package/dist/config/index.js
CHANGED
|
@@ -22,8 +22,7 @@ const defaultConfig = [
|
|
|
22
22
|
languageOptions: {
|
|
23
23
|
globals: {
|
|
24
24
|
...globals.browser,
|
|
25
|
-
...globals.node
|
|
26
|
-
...globals.jest
|
|
25
|
+
...globals.node
|
|
27
26
|
},
|
|
28
27
|
ecmaVersion: "latest",
|
|
29
28
|
sourceType: "module"
|
|
@@ -539,7 +538,8 @@ const defaultConfig = [
|
|
|
539
538
|
"@typescript-eslint/no-unsafe-enum-comparison": "off",
|
|
540
539
|
// Developer may actually mean to use ||
|
|
541
540
|
// Also, refactoring this out may break things
|
|
542
|
-
"@typescript-eslint/prefer-nullish-coalescing": "off"
|
|
541
|
+
"@typescript-eslint/prefer-nullish-coalescing": "off",
|
|
542
|
+
"@typescript-eslint/no-unnecessary-type-conversion": "warn"
|
|
543
543
|
//#endregion
|
|
544
544
|
// TODO: go over https://typescript-eslint.io/linting/troubleshooting/performance-troubleshooting
|
|
545
545
|
}
|
package/dist/config/lumina.js
CHANGED
|
@@ -126,7 +126,7 @@ declare global {
|
|
|
126
126
|
{
|
|
127
127
|
group: ["/@lit/context"],
|
|
128
128
|
importNames: ["ContextProvider", "ContextConsumer", "provide", "consume"],
|
|
129
|
-
message: `For lazy-loading compatibility, import the useContextProvider() and the useContextConsumer() controllers from @arcgis/lumina rather than directly calling the Lit's controllers/decorators
|
|
129
|
+
message: `For lazy-loading compatibility, import the useContextProvider() and the useContextConsumer() controllers from @arcgis/lumina/context rather than directly calling the Lit's controllers/decorators. See https://qawebgis.esri.com/components/lumina/state-management#lit-context`,
|
|
130
130
|
caseSensitive: true
|
|
131
131
|
},
|
|
132
132
|
{
|
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
import { m as makeEslintPlugin } from "../../makePlugin-
|
|
1
|
+
import { m as makeEslintPlugin } from "../../makePlugin-BWc0voYM.js";
|
|
2
2
|
import { AST_NODE_TYPES, ESLintUtils, AST_TOKEN_TYPES } from "@typescript-eslint/utils";
|
|
3
3
|
import ts from "typescript";
|
|
4
4
|
import { camelToKebab } from "@arcgis/components-utils";
|
|
@@ -1277,7 +1277,7 @@ plugin.createRule({
|
|
|
1277
1277
|
return;
|
|
1278
1278
|
}
|
|
1279
1279
|
if (isCreateElementComponent(node)) {
|
|
1280
|
-
return
|
|
1280
|
+
return context.report({
|
|
1281
1281
|
node,
|
|
1282
1282
|
messageId: "default"
|
|
1283
1283
|
});
|
|
@@ -1,4 +1,4 @@
|
|
|
1
|
-
import { m as makeEslintPlugin } from "../../makePlugin-
|
|
1
|
+
import { m as makeEslintPlugin } from "../../makePlugin-BWc0voYM.js";
|
|
2
2
|
import { resolve } from "node:path/posix";
|
|
3
3
|
import { AST_NODE_TYPES } from "@typescript-eslint/utils";
|
|
4
4
|
const plugin = makeEslintPlugin(
|
|
@@ -94,9 +94,10 @@ plugin.createRule({
|
|
|
94
94
|
}
|
|
95
95
|
});
|
|
96
96
|
const reTouchingJsDoc = /\*\/\s+\/\*\*/gu;
|
|
97
|
-
const description$1 = `@arcgis/core imports need to end with .js for better compatibility with @arcgis/core
|
|
97
|
+
const description$1 = `@arcgis/core imports need to end with .js for better compatibility with ESM CDN builds for @arcgis/core and other packages.`;
|
|
98
|
+
const packagesToEnforce = ["@arcgis/core/", "@amcharts/amcharts4/", "@amcharts/amcharts5/"];
|
|
98
99
|
plugin.createRule({
|
|
99
|
-
name: "require-js-in-
|
|
100
|
+
name: "require-js-in-imports",
|
|
100
101
|
meta: {
|
|
101
102
|
docs: {
|
|
102
103
|
description: description$1,
|
|
@@ -119,7 +120,7 @@ plugin.createRule({
|
|
|
119
120
|
return;
|
|
120
121
|
}
|
|
121
122
|
const specifier = node.source.value;
|
|
122
|
-
if (typeof specifier !== "string" || !specifier.startsWith(
|
|
123
|
+
if (typeof specifier !== "string" || !packagesToEnforce.some((pkg) => specifier.startsWith(pkg)) || // Already ends with .js or .json
|
|
123
124
|
specifier.includes(".")) {
|
|
124
125
|
return;
|
|
125
126
|
}
|
package/package.json
CHANGED
|
@@ -1,6 +1,6 @@
|
|
|
1
1
|
{
|
|
2
2
|
"name": "@arcgis/eslint-config",
|
|
3
|
-
"version": "4.34.0-next.
|
|
3
|
+
"version": "4.34.0-next.40",
|
|
4
4
|
"description": "ESLint configuration for arcgis-web-components",
|
|
5
5
|
"type": "module",
|
|
6
6
|
"main": "index.js",
|
|
@@ -20,17 +20,17 @@
|
|
|
20
20
|
],
|
|
21
21
|
"license": "SEE LICENSE IN LICENSE.md",
|
|
22
22
|
"dependencies": {
|
|
23
|
-
"@arcgis/components-utils": "4.34.0-next.
|
|
24
|
-
"@eslint/js": "^9.
|
|
23
|
+
"@arcgis/components-utils": "4.34.0-next.40",
|
|
24
|
+
"@eslint/js": "^9.29.0",
|
|
25
25
|
"@types/confusing-browser-globals": "^1.0.3",
|
|
26
26
|
"confusing-browser-globals": "^1.0.11",
|
|
27
27
|
"eslint-plugin-storybook": "^0.12.0",
|
|
28
28
|
"globals": "^15.12.0",
|
|
29
29
|
"tslib": "^2.8.1",
|
|
30
30
|
"typescript": "~5.8.3",
|
|
31
|
-
"typescript-eslint": "^8.
|
|
31
|
+
"typescript-eslint": "^8.35.0"
|
|
32
32
|
},
|
|
33
33
|
"peerDependencies": {
|
|
34
|
-
"eslint": "^9.
|
|
34
|
+
"eslint": "^9.29.0"
|
|
35
35
|
}
|
|
36
36
|
}
|
/package/dist/plugins/webgis/rules/{require-js-in-core-import.d.ts → require-js-in-imports.d.ts}
RENAMED
|
File without changes
|