Core: refactor 'removeParens' function for DRY

Author: Mersho

src/FSharpLint.Core/Framework/Ast.fs

@@ -45,11 +45,6 @@ module Ast =
         attrs
         |> List.collect (fun attrList -> attrList.Attributes)
 
-    /// Extracts an expression from parentheses e.g. ((x + 4)) -> x + 4
-    let rec removeParens = function
-        | SynExpr.Paren(x, _, _, _) -> removeParens x
-        | x -> x
-
     /// Inlines pipe operators to give a flat function application expression
     /// e.g. `x |> List.map id` to `List.map id x`.
     let (|FuncApp|_|) functionApplication =

src/FSharpLint.Core/Rules/Conventions/FunctionReimplementation/CanBeReplacedWithComposition.fs

@@ -6,6 +6,7 @@ open FSharpLint.Framework.Suggestion
 open FSharp.Compiler.Syntax
 open FSharpLint.Framework.Ast
 open FSharpLint.Framework.Rules
+open FSharpLint.Framework.ExpressionUtilities
 
 let private validateLambdaCannotBeReplacedWithComposition _ lambda range =
     let canBeReplacedWithFunctionComposition expression =

tests/FSharpLint.Core.Tests/Framework/TestAbstractSyntaxArray.fs

@@ -8,6 +8,7 @@ open FSharp.Compiler.Syntax
 open Microsoft.FSharp.Reflection
 open NUnit.Framework
 open TestUtils
+open FSharpLint.Framework.ExpressionUtilities
 
 [<TestFixture>]
 type TestAst() =