Skip to content

Commit 907bebc

Browse files
emypaulson-1993emypaulson-1993
committed
t/code-analyzer/w-20084011/pmd-update-7.18.0
1 parent a7f6b13 commit 907bebc

File tree

15 files changed

+247
-550
lines changed

15 files changed

+247
-550
lines changed

‎package-lock.json‎

Lines changed: 214 additions & 520 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.

‎packages/ENGINE-TEMPLATE/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@
1313
"main": "dist/index.js",
1414
"types": "dist/index.d.ts",
1515
"dependencies": {
16-
"@types/node": "^24.10.1",
16+
"@types/node": "^20.0.0",
1717
"@salesforce/code-analyzer-engine-api": "0.32.0-SNAPSHOT"
1818
},
1919
"devDependencies": {

‎packages/code-analyzer-core/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -17,7 +17,7 @@
1717
"types": "dist/index.d.ts",
1818
"dependencies": {
1919
"@salesforce/code-analyzer-engine-api": "0.32.0-SNAPSHOT",
20-
"@types/node": "^24.10.1",
20+
"@types/node": "^20.0.0",
2121
"csv-stringify": "^6.6.0",
2222
"js-yaml": "^4.1.1",
2323
"semver": "^7.7.3",

‎packages/code-analyzer-engine-api/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -16,7 +16,7 @@
1616
},
1717
"types": "dist/index.d.ts",
1818
"dependencies": {
19-
"@types/node": "^24.10.1"
19+
"@types/node": "^20.0.0"
2020
},
2121
"devDependencies": {
2222
"@eslint/js": "^9.39.1",

‎packages/code-analyzer-eslint-engine/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -22,7 +22,7 @@
2222
"@salesforce/code-analyzer-eslint8-engine": "0.9.0-SNAPSHOT",
2323
"@salesforce/eslint-config-lwc": "^4.1.1",
2424
"@salesforce/eslint-plugin-lightning": "^2.0.0",
25-
"@types/node": "^24.10.1",
25+
"@types/node": "^20.0.0",
2626
"@typescript-eslint/eslint-plugin": "^8.47.0",
2727
"@typescript-eslint/parser": "^8.47.0",
2828
"eslint": "^9.39.1",

‎packages/code-analyzer-eslint-engine/src/base-config.ts‎

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -134,6 +134,10 @@ export class BaseConfigFactory {
134134
return {
135135
...cssConfig,
136136
files: this.engineConfig.file_extensions.css.map(ext => `**/*${ext}`),
137+
// TODO: Remove this workaround once @salesforce-ux/eslint-plugin-slds v1.0.7+ is released.
138+
// The SLDS plugin's CSS config references `language: "css/css"` but doesn't register the
139+
// CSS language plugin in their plugins object. We must add it ourselves to avoid the error:
140+
// "Could not find 'css' in plugin 'css'". See: https://github.com/salesforce-ux/slds-linter/issues/314
137141
plugins: {
138142
...cssConfig.plugins,
139143
css: eslintCss

‎packages/code-analyzer-eslint-engine/src/rule-mappings.ts‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1462,7 +1462,7 @@ export const RULE_MAPPINGS: Record<string, {severity: SeverityLevel, tags: strin
14621462
},
14631463
"@typescript-eslint/no-unused-private-class-members": {
14641464
severity: SeverityLevel.High,
1465-
tags: [COMMON_TAGS.RECOMMENDED, COMMON_TAGS.CATEGORIES.ERROR_PRONE, COMMON_TAGS.LANGUAGES.TYPESCRIPT]
1465+
tags: [COMMON_TAGS.CATEGORIES.ERROR_PRONE, COMMON_TAGS.LANGUAGES.TYPESCRIPT]
14661466
},
14671467
"@typescript-eslint/no-unused-vars": {
14681468
severity: SeverityLevel.High,

‎packages/code-analyzer-eslint-engine/test/test-data/rules_OnlyLwcBaseConfig.goldfile.json‎

Lines changed: 18 additions & 18 deletions
Original file line numberDiff line numberDiff line change
@@ -671,7 +671,7 @@
671671
"description": "Enforce assertion to be made in a test body",
672672
"name": "jest/expect-expect",
673673
"resourceUrls": [
674-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/expect-expect.md"
674+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/expect-expect.md"
675675
],
676676
"severityLevel": 5,
677677
"tags": [
@@ -685,7 +685,7 @@
685685
"description": "Disallow alias methods",
686686
"name": "jest/no-alias-methods",
687687
"resourceUrls": [
688-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-alias-methods.md"
688+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-alias-methods.md"
689689
],
690690
"severityLevel": 3,
691691
"tags": [
@@ -699,7 +699,7 @@
699699
"description": "Disallow commented out tests",
700700
"name": "jest/no-commented-out-tests",
701701
"resourceUrls": [
702-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-commented-out-tests.md"
702+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-commented-out-tests.md"
703703
],
704704
"severityLevel": 5,
705705
"tags": [
@@ -713,7 +713,7 @@
713713
"description": "Disallow calling `expect` conditionally",
714714
"name": "jest/no-conditional-expect",
715715
"resourceUrls": [
716-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-conditional-expect.md"
716+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-conditional-expect.md"
717717
],
718718
"severityLevel": 2,
719719
"tags": [
@@ -727,7 +727,7 @@
727727
"description": "Disallow disabled tests",
728728
"name": "jest/no-disabled-tests",
729729
"resourceUrls": [
730-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-disabled-tests.md"
730+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-disabled-tests.md"
731731
],
732732
"severityLevel": 5,
733733
"tags": [
@@ -741,7 +741,7 @@
741741
"description": "Disallow using a callback in asynchronous tests and hooks",
742742
"name": "jest/no-done-callback",
743743
"resourceUrls": [
744-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-done-callback.md"
744+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-done-callback.md"
745745
],
746746
"severityLevel": 3,
747747
"tags": [
@@ -755,7 +755,7 @@
755755
"description": "Disallow using `exports` in files containing tests",
756756
"name": "jest/no-export",
757757
"resourceUrls": [
758-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-export.md"
758+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-export.md"
759759
],
760760
"severityLevel": 3,
761761
"tags": [
@@ -769,7 +769,7 @@
769769
"description": "Disallow focused tests",
770770
"name": "jest/no-focused-tests",
771771
"resourceUrls": [
772-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-focused-tests.md"
772+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-focused-tests.md"
773773
],
774774
"severityLevel": 3,
775775
"tags": [
@@ -783,7 +783,7 @@
783783
"description": "Disallow identical titles",
784784
"name": "jest/no-identical-title",
785785
"resourceUrls": [
786-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-identical-title.md"
786+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-identical-title.md"
787787
],
788788
"severityLevel": 3,
789789
"tags": [
@@ -797,7 +797,7 @@
797797
"description": "Disallow string interpolation inside snapshots",
798798
"name": "jest/no-interpolation-in-snapshots",
799799
"resourceUrls": [
800-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-interpolation-in-snapshots.md"
800+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-interpolation-in-snapshots.md"
801801
],
802802
"severityLevel": 2,
803803
"tags": [
@@ -811,7 +811,7 @@
811811
"description": "Disallow Jasmine globals",
812812
"name": "jest/no-jasmine-globals",
813813
"resourceUrls": [
814-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-jasmine-globals.md"
814+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-jasmine-globals.md"
815815
],
816816
"severityLevel": 3,
817817
"tags": [
@@ -825,7 +825,7 @@
825825
"description": "Disallow manually importing from `__mocks__`",
826826
"name": "jest/no-mocks-import",
827827
"resourceUrls": [
828-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-mocks-import.md"
828+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-mocks-import.md"
829829
],
830830
"severityLevel": 2,
831831
"tags": [
@@ -839,7 +839,7 @@
839839
"description": "Disallow using `expect` outside of `it` or `test` blocks",
840840
"name": "jest/no-standalone-expect",
841841
"resourceUrls": [
842-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-standalone-expect.md"
842+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-standalone-expect.md"
843843
],
844844
"severityLevel": 3,
845845
"tags": [
@@ -853,7 +853,7 @@
853853
"description": "Require using `.only` and `.skip` over `f` and `x`",
854854
"name": "jest/no-test-prefixes",
855855
"resourceUrls": [
856-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/no-test-prefixes.md"
856+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/no-test-prefixes.md"
857857
],
858858
"severityLevel": 3,
859859
"tags": [
@@ -867,7 +867,7 @@
867867
"description": "Enforce valid `describe()` callback",
868868
"name": "jest/valid-describe-callback",
869869
"resourceUrls": [
870-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/valid-describe-callback.md"
870+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/valid-describe-callback.md"
871871
],
872872
"severityLevel": 2,
873873
"tags": [
@@ -881,7 +881,7 @@
881881
"description": "Enforce valid `expect()` usage",
882882
"name": "jest/valid-expect",
883883
"resourceUrls": [
884-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/valid-expect.md"
884+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/valid-expect.md"
885885
],
886886
"severityLevel": 3,
887887
"tags": [
@@ -895,7 +895,7 @@
895895
"description": "Require promises that have expectations in their chain to be valid",
896896
"name": "jest/valid-expect-in-promise",
897897
"resourceUrls": [
898-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/valid-expect-in-promise.md"
898+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/valid-expect-in-promise.md"
899899
],
900900
"severityLevel": 3,
901901
"tags": [
@@ -909,7 +909,7 @@
909909
"description": "Enforce valid titles",
910910
"name": "jest/valid-title",
911911
"resourceUrls": [
912-
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.0/docs/rules/valid-title.md"
912+
"https://github.com/jest-community/eslint-plugin-jest/blob/v29.2.1/docs/rules/valid-title.md"
913913
],
914914
"severityLevel": 3,
915915
"tags": [

‎packages/code-analyzer-eslint-engine/test/test-data/rules_OnlyTypeScriptBaseConfig.goldfile.json‎

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1042,7 +1042,6 @@
10421042
],
10431043
"severityLevel": 2,
10441044
"tags": [
1045-
"Recommended",
10461045
"ErrorProne",
10471046
"TypeScript"
10481047
]

‎packages/code-analyzer-eslint8-engine/package.json‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -21,7 +21,7 @@
2121
"@salesforce/code-analyzer-engine-api": "0.32.0-SNAPSHOT",
2222
"@salesforce/eslint-config-lwc": "3.7.2",
2323
"@salesforce/eslint-plugin-lightning": "1.0.1",
24-
"@types/node": "^24.10.1",
24+
"@types/node": "^20.0.0",
2525
"@typescript-eslint/eslint-plugin": "8.30.1",
2626
"@typescript-eslint/parser": "8.30.1",
2727
"eslint": "8.57.1",

0 commit comments

Comments
 (0)