Console: enhanced warning reporting

[Mersho]

Improved warning reporting to display line number and column
in the format: fileName.fs(lineNumber, column).

This feature is particularly useful in environments like
VSCode, which highlights these locations and makes them
clickable. When clicked, VSCode will open the corresponding
file at the exact line and position of the cursor.
This functionality allows developers to quickly navigate to the
source of the issue, significantly speeding up the
debugging process.

[Mersho]

Also we could use workflow annotations, but it would cause extra warnings. This is because the "SelfCheck" step runs three times, once for each operating system and we may encounter additional and repetitive warnings.

[knocte]

This is because the "SelfCheck" step runs three times, once for each operating system and we may encounter additional and repetitive warnings.

I don't understand this claim.

[xperiandri]

Rebase, please

[Mersho]

I rebased, and here is what the new warning message looks like:

$ dotnet run --project .\src\FSharpLint.Console\ --framework net9.0 lint C:\Users\PC\source\repos\FSharpLint\tests\FSharpLint.Benchmarks\Benchmark.fs
========== Linting C:\Users\PC\source\repos\FSharpLint\tests\FSharpLint.Benchmarks\Benchmark.fs ==========
`x = true` might be able to be refactored into `x`.
Error on line 14 starting at column 43: C:\Users\PC\source\repos\FSharpLint\tests\FSharpLint.Benchmarks\Benchmark.fs(14,43)
        let checker = FSharpChecker.Create(keepAssemblyContents=true)
                                           ^
See https://fsprojects.github.io/FSharpLint/how-tos/rules/FL0065.html
--------------------------------------------------------------------------------
========== Finished: 1 warnings ==========
========== Summary: 1 warnings ==========

[xperiandri]

But at the beginning of the output, there is a line ========== Linting C:\Users\PC\source\repos\FSharpLint\tests\FSharpLint.Benchmarks\Benchmark.fs ==========
And MSBuild output format switch already defines a very similar format.
I don't think that this change makes sense

Could you compare your new output to the output with -f msbuild switch and check if it meets your needs?

[knocte]

I don't think that this change makes sense

Agreed, sorry.