Remove pipeCall

wcjohnson/lightscript#36
wcjohnson · Oct 7, 2017 · 416c326 · 416c326
1 parent 9578116
commit 416c326
Show file tree

Hide file tree

Showing 12 changed files with 1 addition and 50 deletions.
diff --git a/src/config.lsc b/src/config.lsc
@@ -41,6 +41,7 @@ export getMetadata() ->
       enhancedComprehension: {
         description: "Positional comprehensions and `case` syntax for comprehensions"
         valueType: "boolean"
+        defaultValue: true
         stage: "0"
       }
       noEnforcedSubscriptIndentation: {
@@ -71,10 +72,6 @@ export getMetadata() ->
         description: "Specify an identifier to be transformed into PlaceholderExpression."
         valueType: "string"
       }
-      pipeCall: {
-        description: "a |> b infix binary operator invokes b(a)"
-        valueType: "boolean"
-      }
     }
   }
 
@@ -110,7 +107,6 @@ export getParserOpts(pluginOpts, initialParserOpts) ->
   if pluginOpts?.placeholderArgs: plugins.push("syntacticPlaceholder")
   if pluginOpts?.placeholder:
     parserOpts.placeholder = pluginOpts.placeholder
-  if pluginOpts?.pipeCall: plugins.push("pipeCall")
 
   // TODO: watch upstream on pattern matching; default to their syntax when complete
   // patternMatchingVersion = pluginOpts?.patternMatching or "v4"

diff --git a/src/transforms/pipe.lsc b/src/transforms/pipe.lsc
diff --git a/src/visitors/main.lsc b/src/visitors/main.lsc
@@ -15,7 +15,6 @@ import * as comprehension from "../transforms/comprehension";
 import { transformExistentialExpression } from "../transforms/safe";
 import { maybeReplaceWithInlinedOperator } from "../transforms/inlinedOperators";
 import { transformForInArrayStatement, transformForInObjectStatement, lintForInArrayStatement, lintForInObjectStatement } from "../transforms/for";
-import { transformPipeOperator } from '../transforms/pipe'
 
 import { markIdentifier } from "../state/stdlib";
 
@@ -24,9 +23,6 @@ export default visitMain(compilerState, programPath): void ->
   { opts } = compilerState
   // Main LSC transforms
   programPath.traverse({
-    PipeCallExpression(path): void ->
-      transformPipeOperator(path)
-
     ForInArrayStatement(path): void ->
       // TODO: push the linter conditional down into the transform
       // and get linter status from compilerState

diff --git a/test/fixtures/pipe-call/basic/actual.js b/test/fixtures/pipe-call/basic/actual.js
diff --git a/test/fixtures/pipe-call/basic/expected.js b/test/fixtures/pipe-call/basic/expected.js
diff --git a/test/fixtures/pipe-call/mixin-pattern/actual.js b/test/fixtures/pipe-call/mixin-pattern/actual.js
diff --git a/test/fixtures/pipe-call/mixin-pattern/expected.js b/test/fixtures/pipe-call/mixin-pattern/expected.js
diff --git a/test/fixtures/pipe-call/options.json b/test/fixtures/pipe-call/options.json
diff --git a/test/fixtures/pipe-call/recompose-pattern/actual.js b/test/fixtures/pipe-call/recompose-pattern/actual.js
diff --git a/test/fixtures/pipe-call/recompose-pattern/expected.js b/test/fixtures/pipe-call/recompose-pattern/expected.js
diff --git a/test/fixtures/pipe-call/rhs-expr/actual.js b/test/fixtures/pipe-call/rhs-expr/actual.js
diff --git a/test/fixtures/pipe-call/rhs-expr/expected.js b/test/fixtures/pipe-call/rhs-expr/expected.js