GCop628 false alarm

[sungam3r]

public Task DoSome(int? value) // GCop628: Maybe define this method on 'Nullable' class as it's using 1 of its members (compared to 0 from this type)
{
    if (value.HasValue)
      return Task.CompletedTask;
    return null;
}

Obviously, we cannot move the method inside System.Nullable<T>. The rule should take into account the system types. Also note if you make the method static, then there is no warning.

[Foroughi]

@sungam3r Hi. Its weird b.c I'm not getting the same warning message on the same code. Here is the warning I'm getting :

GCop628: Maybe define this method on 'DoSome' class as it's using 1 of its members (compared to 0 from this type)

Would you please provide the entire class your working on?

[sungam3r]

    <PackageReference Include="GCop.All.Common" Version="2.6.0" PrivateAssets="all" />

using System.Threading.Tasks;

namespace Test
{
    internal class Class1
    {
        public Task DoSome(int? value) // GCop628: Maybe define this method on 'Nullable' class as it's using 1 of its members (compared to 0 from this type)
        {
            if (value.HasValue)
                return Task.CompletedTask;
            return null;
        }
    }
}

[Foroughi]

Even with using the exact version, I couldn't find this issue. Not in unit tests, not in a real scenario.

Real scenario sample :

Unit test result :

@marjanjavid @Karvan Would you please you guys have a look if you can find the issue on this one ?
@sungam3r Would you please share a light version of your repo?

[sungam3r]

ConsoleApp7.zip