From daf2a27182f272d2370d7b28c38145e856f50c0b Mon Sep 17 00:00:00 2001 From: Umur Ozkul Date: Thu, 14 Apr 2022 02:40:59 +0200 Subject: [PATCH] PR#226, add missing variants to switch statements Code is passing unit tests however there were missing variands. Fixed --- .../Reducer/Reducer_Expression/Reducer_Expression.res | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/packages/squiggle-lang/src/rescript/Reducer/Reducer_Expression/Reducer_Expression.res b/packages/squiggle-lang/src/rescript/Reducer/Reducer_Expression/Reducer_Expression.res index 8498c294..d599fe7f 100644 --- a/packages/squiggle-lang/src/rescript/Reducer/Reducer_Expression/Reducer_Expression.res +++ b/packages/squiggle-lang/src/rescript/Reducer/Reducer_Expression/Reducer_Expression.res @@ -119,7 +119,8 @@ let rec reduceExpression = (expression: t, bindings: T.bindings): result => switch expression { - | T.EValue(value) => expression->Ok + | T.EValue(_value) => expression->Ok + | T.EBindings(_value) => expression->Ok | T.EList(list) => { let racc: result, 'e> = list->Belt.List.reduceReverse(Ok(list{}), ( racc, @@ -155,6 +156,7 @@ let rec reduceExpression = (expression: t, bindings: T.bindings): resultResult.flatMap(acc => acc->reduceValueList) } + | _ => RETodo("Error: Bindings cannot be reduced to values")->Error } let rExpandedExpression: result = expression->seekMacros(bindings)