search

jmecosta / sonar-fsharp-plugin

F# SonarQube (TM) plugin - support for F#
GNU Lesser General Public License v3.0
52 stars 14 forks source link

Not Supported member Report #68

Open milbrandt opened 5 years ago

milbrandt commented 5 years ago

1.1.0-166-rc: reports on 2 files in a MiddlewareStatus repo:

2019-06-22T19:34:49.9139899Z 21:34:49.898 INFO: Not Supported member Report 2019-06-22T19:34:50.5440443Z 21:34:50.539 INFO: Lint Ok

Originally posted by @milbrandt in https://github.com/jmecsoftware/sonar-fsharp-plugin/pull/30#issuecomment-504693671

source\WindowsService\ServiceInstaller.fs source\WindowsService\WindowsService.fs

milbrandt commented 5 years ago

This report I could observe also on other builds.

jmecosta commented 5 years ago

i also have those ones! Looks that needs to be implemented in the core side

milbrandt commented 5 years ago

running dotnet-fsharplint from commandline is running fine on the same files, reporting 0 warnings. Running FsSonarRunner.exe on These files separatly report the above not supported member report.

milbrandt commented 5 years ago

Issue is reported on lines with inherite like inherit ServiceBase()

milbrandt commented 5 years ago

Code in UntypedAstUtils.fsborrowed from deprecated Visual F# Power Tools which starting with Visual Studio 2017 is ported to Visual F# Tools

milbrandt commented 5 years ago

visitMember only support 4 DU cases of SynMemberDefn in the code borrowed from \src\FSharp.Editing\Common\UntypedAstUtils.fs. 11 cases exist. Not critical, Plugin should work even with this.