Skip to content
Merged
7 changes: 3 additions & 4 deletions package.json
Original file line number Diff line number Diff line change
Expand Up @@ -8,8 +8,8 @@
"concurrently": "9.1.0",
"husky": "^9.1.6",
"typescript": "^5.7.3",
"vitest": "^3.0.7",
"@vitest/ui": "^3.0.7",
"vitest": "^3.1.2",
"@vitest/ui": "^3.1.2",
"lint-staged": "^15.4.3",
"eslint": "^9.23.0",
"eslint-config-prettier": "^10.1.1",
Expand All @@ -26,8 +26,7 @@
"typescript-eslint": "^8.29.0",
"@typescript-eslint/eslint-plugin": "^8.29.0",
"@typescript-eslint/parser": "^8.29.0",
"@linode/eslint-plugin-cloud-manager": "^0.0.10",
"jiti": "^2.4.2"
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Oh?! I thought i had removed it

"@linode/eslint-plugin-cloud-manager": "^0.0.10"
},
"scripts": {
"lint:all": "pnpm -r --parallel lint",
Expand Down
5 changes: 2 additions & 3 deletions packages/manager/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -171,10 +171,9 @@
"@types/recompose": "^0.30.0",
"@types/redux-mock-store": "^1.0.1",
"@types/throttle-debounce": "^1.0.0",
"@types/uuid": "^3.4.3",
"@types/zxcvbn": "^4.4.0",
"@vitejs/plugin-react-swc": "^3.7.2",
"@vitest/coverage-v8": "^3.0.7",
"@vitest/coverage-v8": "^3.1.2",
"axe-core": "^4.10.2",
"chai-string": "^1.5.0",
"concurrently": "^9.1.0",
Expand All @@ -199,7 +198,7 @@
"redux-mock-store": "^1.5.3",
"storybook": "^8.6.7",
"storybook-dark-mode": "4.0.1",
"vite": "^6.2.5",
"vite": "^6.3.2",
"vite-plugin-svgr": "^3.2.0"
},
"browserslist": [
Expand Down
Original file line number Diff line number Diff line change
@@ -1,9 +1,18 @@
import { Autocomplete } from '@linode/ui';
import React from 'react';

import { engineOptions } from './constants';
import type { AlertFilterKey, EngineType } from './types';

import type { AlertFilterKey } from './types';
const engineOptions: EngineType[] = [
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I made this change because there was a circular dependency here causing a unit test failure

packages/manager/src/features/CloudPulse/Alerts/AlertsResources/AlertsEngineTypeFilter.tsx ➑️ packages/manager/src/features/CloudPulse/Alerts/AlertsResources/constants.ts ➑️ packages/manager/src/features/CloudPulse/Alerts/AlertsResources/AlertsEngineTypeFilter.tsx

{
id: 'mysql',
label: 'MySQL',
},
{
id: 'postgresql',
label: 'PostgreSQL',
},
];

export interface AlertsEngineOptionProps {
/**
Expand Down
Original file line number Diff line number Diff line change
Expand Up @@ -10,7 +10,6 @@ import type { AlertInstance } from './DisplayAlertResources';
import type { TextWithInfoProp } from './TextWithExtraInfo';
import type {
AlertAdditionalFilterKey,
EngineType,
ServiceColumns,
ServiceFilterConfig,
} from './types';
Expand Down Expand Up @@ -97,17 +96,6 @@ export const alertAdditionalFilterKeyMap: Record<
tags: 'tags',
};

export const engineOptions: EngineType[] = [
{
id: 'mysql',
label: 'MySQL',
},
{
id: 'postgresql',
label: 'PostgreSQL',
},
];

export const databaseTypeClassMap: Record<DatabaseTypeClass, string> = {
dedicated: 'dedicated',
nanode: 'nanode',
Expand Down
2 changes: 1 addition & 1 deletion packages/search/package.json
Original file line number Diff line number Diff line change
Expand Up @@ -16,6 +16,6 @@
"@linode/api-v4": "workspace:*"
},
"peerDependencies": {
"vite": "^6.2.5"
"vite": "^6.3.2"
}
}
Loading