Allow Kernel unary ! and not as valid pipe starts. Closes #183

CHANGELOG.md

@@ -8,6 +8,7 @@ they can and will change without that change being reflected in Styler's semanti
 ### Fixes
 
 * fix `with` arrow replacement + redundant body removal creating invalid statements (#184, h/t @JesseHerrick)
+* allow Kernel unary `!` and `not` as valid pipe starts (#183, h/t @nherzing)
 
 ## 1.0.0-rc.2
 

lib/style/pipes.ex

@@ -35,7 +35,7 @@ defmodule Styler.Style.Pipes do
   # most of these values were lifted directly from credo's pipe_chain_start.ex
   @literal ~w(__block__ __aliases__ unquote)a
   @value_constructors ~w(% %{} .. ..// <<>> @ {} ^ & fn from)a
-  @kernel_ops ~w(++ -- && || in - * + / > < <= >= == and or != !== === <>)a
+  @kernel_ops ~w(++ -- && || in - * + / > < <= >= == and or != !== === <> ! not)a
   @special_ops ~w(||| &&& <<< >>> <<~ ~>> <~ ~> <~>)a
   @special_ops @literal ++ @value_constructors ++ @kernel_ops ++ @special_ops
 

test/style/pipes_test.exs

@@ -435,6 +435,13 @@ defmodule Styler.Style.PipesTest do
   end
 
   describe "valid pipe starts & unpiping" do
+    test "kernel unary starts" do
+      assert_style("!b |> c() |> d()", "!b |> c() |> d()")
+      assert_style("!x(b) |> c() |> d()")
+      assert_style("not(b) |> c() |> d()", "not b |> c() |> d()")
+      assert_style("not x(b) |> c() |> d()")
+    end
+
     test "writes brackets for unpiped kwl" do
       assert_style("foo(kwl: :arg) |> bar()", "[kwl: :arg] |> foo() |> bar()")
       assert_style("%{a: foo(a: :b, c: :d) |> bar()}", "%{a: [a: :b, c: :d] |> foo() |> bar()}")