You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
Running F# Lint to its own source-code is not passing.
This makes me think some of the default rules are not good, if the linter itself doesn't obey those rules.
So consider turning the hitting rules off by default.
Repro steps
Clone this repo, build, and run:
C:\git\FSharpLint\src\FSharpLint.Console>dotnet run --framework net5.0 lint c:\git\FSharpLint\src\FSharpLint.Core\FSharpLint.Core.fsproj
Expected behavior
Pass with no complaints.
Actual behavior
Mostly it will hit just generic argument not being in PascalCase:
Consider changing `t` to PascalCase.
Error on line 87 starting at column 16
type Result<'t> =
^
Consider changing `key` to PascalCase.
Error on line 18 starting at column 47
let addOrUpdate key value (dict:Dictionary<'key,'value>) =
^
Would default camelCase be ok for type-arguments?
Related information
Operating system Windows 11
Branch master
.NET Runtime, CoreCLR or Mono Version .NET 5
The text was updated successfully, but these errors were encountered:
Description
Running F# Lint to its own source-code is not passing.
This makes me think some of the default rules are not good, if the linter itself doesn't obey those rules.
So consider turning the hitting rules off by default.
Repro steps
Clone this repo, build, and run:
Expected behavior
Pass with no complaints.
Actual behavior
Mostly it will hit just generic argument not being in PascalCase:
Would default camelCase be ok for type-arguments?
Related information
The text was updated successfully, but these errors were encountered: