Skip to content

Commit

Permalink
Upgrade eslint
Browse files Browse the repository at this point in the history
jgerigmeyer committed Oct 8, 2024
1 parent 88771bb commit 77e1cf7
Showing 7 changed files with 426 additions and 718 deletions.
8 changes: 0 additions & 8 deletions .eslintignore

This file was deleted.

31 changes: 0 additions & 31 deletions .eslintrc.cjs

This file was deleted.

3 changes: 0 additions & 3 deletions .github/dependabot.yml
Original file line number Diff line number Diff line change
@@ -25,6 +25,3 @@ updates:
- 'patch'
ignore:
- dependency-name: '@playwright/test'
- dependency-name: 'eslint'
update-types:
- 'version-update:semver-major'
62 changes: 62 additions & 0 deletions eslint.config.js
Original file line number Diff line number Diff line change
@@ -0,0 +1,62 @@
/* eslint-disable import/no-named-as-default-member */

import js from '@eslint/js';
import prettier from 'eslint-config-prettier';
import importPlugin from 'eslint-plugin-import';
import simpleImportSort from 'eslint-plugin-simple-import-sort';
import globals from 'globals';
import tseslint from 'typescript-eslint';

export default [
{
ignores: [
'.git/*',
'.vscode/*',
'coverage/*',
'dist/*',
'node_modules/*',
'playwright-report/*',
],
},
js.configs.recommended,
...tseslint.configs.recommended,
...tseslint.configs.stylistic,
importPlugin.flatConfigs.recommended,
importPlugin.flatConfigs.typescript,
prettier,
{
files: ['**/*.{js,mjs,cjs,ts,cts,mts}'],
languageOptions: {
globals: {
...globals.node,
...globals.es2021,
},
},
plugins: { 'simple-import-sort': simpleImportSort },
settings: {
'import/resolver': {
typescript: {
project: 'tsconfig.json',
},
},
},
rules: {
'no-warning-comments': ['warn', { terms: ['todo', 'fixme', '@@@'] }],
'simple-import-sort/imports': 'warn',
'simple-import-sort/exports': 'warn',
'import/first': 'warn',
'import/newline-after-import': 'warn',
'import/no-duplicates': ['error', { 'prefer-inline': true }],
'import/order': 'off',
},
},
{
files: ['src/**/*.{js,mjs,cjs,ts,cts,mts}'],
languageOptions: {
globals: {
...globals.browser,
...globals.es2021,
},
},
},
];
1,024 changes: 357 additions & 667 deletions package-lock.json

Large diffs are not rendered by default.

13 changes: 7 additions & 6 deletions package.json
Original file line number Diff line number Diff line change
@@ -86,22 +86,23 @@
"prepack": "npm run build"
},
"devDependencies": {
"@eslint/js": "^9.12.0",
"@playwright/test": "1.33.0",
"@types/eslint__js": "^8.42.3",
"@types/eslint-config-prettier": "^6.11.3",
"@types/node": "*",
"@typescript-eslint/eslint-plugin": "^8.8.0",
"@typescript-eslint/parser": "^8.8.0",
"esbuild": "^0.24.0",
"eslint": "^8.57.1",
"eslint": "^9.12.0",
"eslint-config-prettier": "^9.1.0",
"eslint-import-resolver-typescript": "^3.6.3",
"eslint-plugin-import": "^2.30.0",
"eslint-plugin-import": "^2.31.0",
"eslint-plugin-simple-import-sort": "^12.1.1",
"husky": "^9.1.6",
"lint-staged": "^15.2.10",
"npm-run-all": "^4.1.5",
"prettier": "3.3.3",
"ts-node": "^10.9.2",
"typescript": "^5.6.2"
"typescript": "^5.6.2",
"typescript-eslint": "^8.8.1"
},
"lint-staged": {
"*.{js,ts}": [
3 changes: 0 additions & 3 deletions src/popover-helpers.ts
Original file line number Diff line number Diff line change
@@ -84,7 +84,6 @@ function topMostAutoPopover(document: Document): HTMLElement | null {
const documentPopovers = autoPopoverList.get(document);
for (const popover of documentPopovers || []) {
if (!popover.isConnected) {
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
documentPopovers!.delete(popover);
} else {
return popover;
@@ -285,14 +284,12 @@ export function showPopover(element: HTMLElement) {
if (!topLayerElements.has(document)) {
topLayerElements.set(document, new Set());
}
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
topLayerElements.get(document)!.add(element);
popoverFocusingSteps(element);
if (element.popover === 'auto') {
if (!autoPopoverList.has(document)) {
autoPopoverList.set(document, new Set());
}
// eslint-disable-next-line @typescript-eslint/no-non-null-assertion
autoPopoverList.get(document)!.add(element);
setInvokerAriaExpanded(popoverInvoker.get(element), true);
}

0 comments on commit 77e1cf7

Please sign in to comment.