Obsolete warning isn't produced for some obsolete symbol usages

[Obsolete("Class is obsolete")]
public class ObsoleteClass
{
}

let c1: ObsoleteClass = null
let c2 = ObsoleteClass()

Two warnings are expected, but only one is produced:

C# compiler produces two warnings:

ObsoleteClass c = new ObsoleteClass();

[edgarfgp]

@auduchinok I believe I have covered all the cases in this issue including events 😀See https://github.com/dotnet/fsharp/blob/main/tests/FSharp.Compiler.ComponentTests/Language/ObsoleteAttributeCheckingTests.fs

[edgarfgp]

@vzarytovskii @auduchinok

Is it on the main, or latest stable? I remember @edgarfgp was doing some changes there?

I think is in main

It’s in 6.0.301 and in FCS from ~March.

My changes where June 13 https://github.com/dotnet/fsharp/pull/13257 (See the testing scenarios to see this is fixed) And it did address some of the inconsistencies . But there are still some pending like https://github.com/dotnet/fsharp/issues/6628 and https://github.com/dotnet/fsharp/issues/11529

let c1: ObsoleteClass = null
let c2 = ObsoleteClass()

Try adding new before the class name ?

let c2 = new ObsoleteClass()

I think we can add more testing to validate this behaviours and see what is still missing ?