Merge pull request #46 from perses/antoinethebaud/timeserieschart-adj…

…ust-migrate [IGNORE] TimeSeriesChart: mystery fix for perses/perses migration unit tests
perses · Feb 7, 2025 · 5a5b3cd · 5a5b3cd
2 parents ec9ff33 + 3f24d98
commit 5a5b3cd
Show file tree

Hide file tree

Showing 3 changed files with 13 additions and 9 deletions.
diff --git a/TimeSeriesChart/package.json b/TimeSeriesChart/package.json
@@ -1,7 +1,7 @@
 {
   "name": "@perses-dev/timeseries-chart",
   "private": true,
-  "version": "0.4.1",
+  "version": "0.4.2",
   "scripts": {
     "dev": "rsbuild dev",
     "build": "rsbuild build",

diff --git a/TimeSeriesChart/schemas/migrate/migrate.cue b/TimeSeriesChart/schemas/migrate/migrate.cue
@@ -101,11 +101,15 @@ spec: {
 	// visual
 	#lineWidth: *#panel.fieldConfig.defaults.custom.lineWidth | null
 	if #lineWidth != null {
-		visual: lineWidth: [// switch
-			if #lineWidth > 3 {3},       // line width can't go beyond 3 in Perses
-			if #lineWidth < 0.25 {0.25}, // line width can't go below 0.25 in Perses
-			#lineWidth,
-		][0]
+		if #lineWidth > 3 {
+			visual: lineWidth: 3 // line width can't go beyond 3 in Perses
+		}
+		if #lineWidth < 0.25 {
+			visual: lineWidth: 0.25 // line width can't go below 0.25 in Perses
+		}
+		if #lineWidth >= 0.25 && #lineWidth <= 3 {
+			visual: lineWidth: #lineWidth
+		}
 	}
 
 	#fillOpacity: *#panel.fieldConfig.defaults.custom.fillOpacity | null

diff --git a/package-lock.json b/package-lock.json