Skip to content

Commit 59983f2

Browse files
matticbotrenovate-botmanzoorwanijk
authored
Update Eslint packages (#45658)
* Fix `testing-library/no-unnecessary-act` * Disable "package-json/require-license" for package.json files. * Fix unit tests --------- Co-authored-by: Renovate Bot <[email protected]> Co-authored-by: Manzoor Wani <[email protected]>
1 parent baab46a commit 59983f2

File tree

10 files changed

+290
-346
lines changed

10 files changed

+290
-346
lines changed

package.json

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -30,7 +30,7 @@
3030
"version-packages": "bash ./tools/version-packages.sh"
3131
},
3232
"devDependencies": {
33-
"eslint": "9.35.0",
33+
"eslint": "9.38.0",
3434
"husky": "9.1.7",
3535
"jetpack-cli": "workspace:*",
3636
"jetpack-js-tools": "workspace:*",

pnpm-lock.yaml

Lines changed: 259 additions & 323 deletions
Some generated files are not rendered by default. Learn more about customizing how changed files appear on GitHub.
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: patch
2+
Type: fixed
3+
4+
Fix lints.

projects/js-packages/charts/src/components/pie-semi-circle-chart/test/pie-semi-circle-chart.test.tsx

Lines changed: 3 additions & 10 deletions
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
import { render, screen, waitFor } from '@testing-library/react';
22
import userEvent from '@testing-library/user-event';
3-
import { act } from 'react';
43
import { GlobalChartsProvider } from '../../../providers';
54
import PieSemiCircleChart from '../pie-semi-circle-chart';
65

@@ -67,9 +66,7 @@ describe( 'PieSemiCircleChart', () => {
6766
const firstSegment = segments[ 0 ];
6867

6968
// Wrap hover interaction in act()
70-
await act( async () => {
71-
await user.hover( firstSegment );
72-
} );
69+
await user.hover( firstSegment );
7370

7471
// Check for tooltip by looking for the specific tooltip role
7572
const tooltip = screen.getByRole( 'tooltip' );
@@ -90,17 +87,13 @@ describe( 'PieSemiCircleChart', () => {
9087
const segments = screen.getAllByTestId( 'pie-segment' );
9188
const firstSegment = segments[ 0 ];
9289

93-
await act( async () => {
94-
await user.hover( firstSegment );
95-
} );
90+
await user.hover( firstSegment );
9691

9792
// Wait for tooltip to be visible - it should show in the BaseTooltip component
9893
const tooltip = await screen.findByRole( 'tooltip' );
9994
expect( tooltip ).toHaveTextContent( 'MacOS' );
10095

101-
await act( async () => {
102-
await user.unhover( firstSegment );
103-
} );
96+
await user.unhover( firstSegment );
10497

10598
// Verify tooltip is gone - checking for the tooltip role specifically
10699
await waitFor( () => {
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: patch
2+
Type: fixed
3+
4+
Fix lints.

projects/js-packages/eslint-changed/tests/cli.test.js

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -339,7 +339,7 @@ describe( 'bin/eslint-changed.js', () => {
339339
const usedDeprecatedRules = [
340340
{
341341
info: {
342-
availableUntil: '10.0.0',
342+
availableUntil: '11.0.0',
343343
deprecatedSince: '8.53.0',
344344
message: 'Formatting rules are being moved out of ESLint core.',
345345
replacedBy: [
@@ -363,7 +363,7 @@ describe( 'bin/eslint-changed.js', () => {
363363
},
364364
{
365365
info: {
366-
availableUntil: '10.0.0',
366+
availableUntil: '11.0.0',
367367
deprecatedSince: '8.53.0',
368368
message: 'Formatting rules are being moved out of ESLint core.',
369369
replacedBy: [
Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,4 @@
1+
Significance: patch
2+
Type: changed
3+
4+
Update package dependencies.

projects/js-packages/eslint-config-target-es/package.json

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -45,8 +45,8 @@
4545
},
4646
"devDependencies": {
4747
"@wordpress/browserslist-config": "6.31.0",
48-
"eslint": "9.35.0",
49-
"eslint-plugin-es-x": "9.1.0",
48+
"eslint": "9.38.0",
49+
"eslint-plugin-es-x": "9.1.2",
5050
"globals": "16.0.0",
5151
"jest": "30.0.4"
5252
},

tools/js-tools/eslintrc/base.mjs

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -477,6 +477,9 @@ export function makeBaseConfig( configurl, opts = {} ) {
477477

478478
// Maybe someday, but not yet.
479479
'package-json/require-type': 'off',
480+
481+
// Not needed for every package.json.
482+
'package-json/require-license': 'off',
480483
},
481484
},
482485
{

tools/js-tools/package.json

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -14,9 +14,9 @@
1414
"@babel/core": "7.28.4",
1515
"@babel/preset-react": "7.27.1",
1616
"@babel/preset-typescript": "7.27.1",
17-
"@eslint/compat": "1.3.2",
17+
"@eslint/compat": "1.4.0",
1818
"@eslint/eslintrc": "3.3.1",
19-
"@eslint/js": "9.35.0",
19+
"@eslint/js": "9.38.0",
2020
"@eslint/json": "0.13.2",
2121
"@octokit/auth-token": "5.1.2",
2222
"@octokit/rest": "21.1.1",
@@ -29,25 +29,25 @@
2929
"chalk": "5.6.2",
3030
"debug": "4.4.1",
3131
"enquirer": "2.4.1",
32-
"eslint": "9.35.0",
32+
"eslint": "9.38.0",
3333
"eslint-config-prettier": "10.1.8",
3434
"eslint-import-resolver-typescript": "4.4.4",
3535
"eslint-json-compat-utils": "0.2.1",
36-
"eslint-plugin-es-x": "9.1.0",
36+
"eslint-plugin-es-x": "9.1.2",
3737
"eslint-plugin-import": "2.32.0",
3838
"eslint-plugin-jest": "29.0.1",
3939
"eslint-plugin-jest-dom": "5.5.0",
4040
"eslint-plugin-jsdoc": "51.4.1",
4141
"eslint-plugin-jsx-a11y": "6.10.2",
4242
"eslint-plugin-lodash": "8.0.0",
4343
"eslint-plugin-n": "17.23.1",
44-
"eslint-plugin-package-json": "0.56.3",
44+
"eslint-plugin-package-json": "0.59.0",
4545
"eslint-plugin-playwright": "2.2.2",
4646
"eslint-plugin-prettier": "5.5.4",
4747
"eslint-plugin-react": "7.37.5",
4848
"eslint-plugin-react-hooks": "5.2.0",
49-
"eslint-plugin-svelte": "3.12.3",
50-
"eslint-plugin-testing-library": "7.8.0",
49+
"eslint-plugin-svelte": "3.12.5",
50+
"eslint-plugin-testing-library": "7.13.3",
5151
"eslint-plugin-you-dont-need-lodash-underscore": "6.14.0",
5252
"glob": "11.0.2",
5353
"globals": "16.0.0",
@@ -67,7 +67,7 @@
6767
"svelte": "4.2.19",
6868
"svelte-eslint-parser": "1.1.3",
6969
"typescript": "5.9.3",
70-
"typescript-eslint": "8.44.0",
70+
"typescript-eslint": "8.46.2",
7171
"yaml": "2.2.2"
7272
}
7373
}

0 commit comments

Comments
 (0)