Skip to content

chore: Using prepare package lock script from build tools #155

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
pan-kot merged 1 commit into from
Feb 3, 2026
Merged

chore: Using prepare package lock script from build tools #155

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
1 change: 1 addition & 0 deletions .eslintrc
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -84,6 +84,7 @@
],
"simple-import-sort/imports": "warn",
"@vitest/no-focused-tests": "error",
"@vitest/no-conditional-expect": "off",
},
"settings": {
"react": {
Expand Down
16,309 changes: 2,847 additions & 13,462 deletions package-lock.json
View file
Open in desktop

Large diffs are not rendered by default.

6 changes: 3 additions & 3 deletions package.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -116,13 +116,13 @@
"stylelint": "^16.9.0",
"stylelint-config-recommended-scss": "^14.1.0",
"stylelint-prettier": "^5.0.2",
"typescript": "^4.9.4",
"typescript": "^5.9.2",
"vite": "^6.4.1",
"vitest": "^3.0.7"
},
"//": "ensure that typedoc uses latest typescript. It prints a warning, but works",
"overrides": {
"typescript": "^4.9.4",
"typescript": "^5.9.2",
Copy link
Member Author

@pan-kot pan-kot Jan 27, 2026

Choose a reason for hiding this comment

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

This matches the version of TS used in the main components package. It fixes a compilation error with test-utils, present after upgrading the deps.

"glob": "^10.5.0"
},
"lint-staged": {
Expand All @@ -133,7 +133,7 @@
"stylelint --fix"
],
"package-lock.json": [
"./scripts/prepare-package-lock.js"
"prepare-package-lock"
]
}
}
51 changes: 0 additions & 51 deletions scripts/prepare-package-lock.js
View file
Open in desktop

This file was deleted.

12 changes: 6 additions & 6 deletions src/__tests__/__snapshots__/documenter.test.ts.snap
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -269,7 +269,7 @@ for the user to recover from the error.",
},
{
"inlineType": {
"name": "NonCancelableEventHandler<{}>",
"name": "NonCancelableEventHandler",
Copy link
Member Author

@pan-kot pan-kot Jan 27, 2026

Choose a reason for hiding this comment

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

This is caused by TS version upgrade.

"parameters": [
{
"name": "event",
Expand All @@ -281,7 +281,7 @@ for the user to recover from the error.",
},
"name": "onRecoveryClick",
"optional": true,
"type": "NonCancelableEventHandler<{}>",
"type": "NonCancelableEventHandler",
},
{
"inlineType": {
Expand Down Expand Up @@ -923,7 +923,7 @@ for the user to recover from the error.",
},
{
"inlineType": {
"name": "NonCancelableEventHandler<{}>",
"name": "NonCancelableEventHandler",
"parameters": [
{
"name": "event",
Expand All @@ -935,7 +935,7 @@ for the user to recover from the error.",
},
"name": "onRecoveryClick",
"optional": true,
"type": "NonCancelableEventHandler<{}>",
"type": "NonCancelableEventHandler",
},
{
"inlineType": {
Expand Down Expand Up @@ -1718,7 +1718,7 @@ for the user to recover from the error.",
},
{
"inlineType": {
"name": "NonCancelableEventHandler<{}>",
"name": "NonCancelableEventHandler",
"parameters": [
{
"name": "event",
Expand All @@ -1730,7 +1730,7 @@ for the user to recover from the error.",
},
"name": "onRecoveryClick",
"optional": true,
"type": "NonCancelableEventHandler<{}>",
"type": "NonCancelableEventHandler",
},
{
"inlineType": {
Expand Down
4 changes: 1 addition & 3 deletions src/cartesian-chart/interfaces.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -8,9 +8,7 @@ import { NonCancelableEventHandler } from "../internal/events";
// we pass options directly to the component, instead of grouping them all into a single "options" property.
// We do still organize related options in groups, e.g.: "SeriesOptions", "TooltipOptions".
export interface CartesianChartProps
extends CoreTypes.BaseChartOptions,
CoreTypes.CoreCartesianOptions,
CoreTypes.WithCartesianI18nStrings {
extends CoreTypes.BaseChartOptions, CoreTypes.CoreCartesianOptions, CoreTypes.WithCartesianI18nStrings {
/**
* Inverts X and Y axes. Use it to show horizontal columns (bars).
* This property corresponds to [chart.inverted](https://api.highcharts.com/highcharts/chart.inverted).
Expand Down
3 changes: 2 additions & 1 deletion src/core/interfaces.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -314,7 +314,8 @@ export interface CoreCartesianOptions {
}

export interface CoreChartProps
extends Pick<
extends
Pick<
BaseChartOptions,
| "highcharts"
| "fallback"
Expand Down
4 changes: 4 additions & 0 deletions src/core/utils.ts
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -215,6 +215,10 @@ export function getVisibleLegendItems(options: Highcharts.Options) {
};

options.series?.forEach((series) => {
if (!("type" in series)) {
Copy link
Member Author

@pan-kot pan-kot Jan 27, 2026

Choose a reason for hiding this comment

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

The Highcharts series option type was extended with UnknownSeriesOptionsType, which does not include the type field.

Ref: highcharts/highcharts-declarations-generator@b381948#diff-027cb4cef6330a88a19cf87488bbabc97c3ebd9a9e5a2be19084339eabc5ff39R305

return;
}

// The pie series is not shown in the legend. Instead, we show pie segments (points).
if (series.type === "pie") {
if (Array.isArray(series.data)) {
Expand Down
2 changes: 1 addition & 1 deletion tsconfig.e2e.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -6,6 +6,6 @@
"types": ["vite/client"],
"tsBuildInfoFile": "./.cache/e2e-tests.tsbuildinfo"
},
"include": ["types", "test"],
"include": ["types", "test", "vite/client"],
Copy link
Member Author

@pan-kot pan-kot Jan 27, 2026

Choose a reason for hiding this comment

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

This fixes import.meta.glob call in visual tests. W/o these types, the call to .glob fails.

"exclude": []
}
Loading