mirror of
https://github.com/actualbudget/actual.git
synced 2026-04-28 18:40:34 -05:00
[AI] Restore @actual-app/web no-restricted-imports rule and fix stacked disable comments
Keep the existing no-restricted-imports rule for @actual-app/web in loot-core as-is alongside the new no-cross-package-imports rule. Fix the stacked oxlint-disable-next-line comments in desktop-electron/e2e/onboarding.test.ts by combining them into a single line so both suppressions apply correctly. https://claude.ai/code/session_01XjmtRs1P9Rg7FNJAYVcaZJ
This commit is contained in:
@@ -351,11 +351,14 @@
|
||||
{
|
||||
"group": ["**/style/themes/*"],
|
||||
"message": "Please do not import theme files directly"
|
||||
},
|
||||
{
|
||||
"group": ["@actual-app/web/**/*"],
|
||||
"message": "Please do not import `@actual-app/web` in `loot-core`"
|
||||
}
|
||||
]
|
||||
}
|
||||
],
|
||||
"actual/no-cross-package-imports": "error",
|
||||
"eslint/no-useless-constructor": "error",
|
||||
"eslint/no-undef": "error",
|
||||
"eslint/no-unused-expressions": "error"
|
||||
|
||||
@@ -1,5 +1,4 @@
|
||||
// oxlint-disable-next-line eslint/no-restricted-imports -- fix me
|
||||
// oxlint-disable-next-line actual/no-cross-package-imports -- fix me
|
||||
// oxlint-disable-next-line eslint/no-restricted-imports, actual/no-cross-package-imports -- fix me
|
||||
import { ConfigurationPage } from '@actual-app/web/e2e/page-models/configuration-page';
|
||||
import { expect } from '@playwright/test';
|
||||
|
||||
|
||||
Reference in New Issue
Block a user