Skip to content

Conversation

@webwarrior-ws
Copy link
Contributor

Worked around FCS bug that prevented linting src/FSharpLint.Core/Application/Configuration.fs file.

In src/FSharpLint.Core/Application/Configuration.fs to avoid FSC bug
that leads to error:
```
Failed to parse file D:\Projects\work\FSharpLint\src\FSharpLint.Core\Application\Configuration.fs
Exception Message:
The exception has been reported. This internal exception should now be caught at an error recovery point on the stack. Original message: ConstraintSolverMissingConstraint
  ({ includeStaticParametersInTypeNames = false
     openTopPathsSorted =
      Internal.Utilities.Library.InterruptibleLazy`1[Microsoft.FSharp.Collections.FSharpList`1[Microsoft.FSharp.Collections.FSharpList`1[System.String]]]
     openTopPathsRaw = []
     shortTypeNames = false
     suppressNestedTypes = false
     maxMembers = None
     showObsoleteMembers = false
     showHiddenMembers = false
     showTyparBinding = false
     showInferenceTyparAnnotations = false
     suppressInlineKeyword = true
     suppressMutableKeyword = false
     showMemberContainers = false
     shortConstraints = false
     useColonForReturnType = false
     showAttributes = false
     showCsharpCodeAnalysisAttributes = false
     showOverrides = true
     showStaticallyResolvedTyparAnnotations = true
     showNullnessAnnotations = None
     abbreviateAdditionalConstraints = false
     showTyparDefaultConstraints = false
     showDocumentation = false
     shrinkOverloads = true
     printVerboseSignatures = false
     escapeKeywordNames = false
     g = <TcGlobals>
     contextAccessibility = public
     generatedValueLayout = <fun:Empty@3228>
     genericParameterStyle = Implicit }, Config,
   SupportsComparison (139,4--139,10), (139,4--139,10), (139,4--139,10)))
```
Also added FSharpx.Collections library because
NoPartialFunctions rule recommends tryHeadTail function from
this library instead of tail functions.
By making tryFindTypedExpression function tail-recursive.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants