SuppressNullableWarningExpression should break consistently with invocation expressions

[belav]

closes #596

[belav]

Some edge cases from https://github.com/belav/csharpier-repos/pull/30/files

                    NamingStrategy namingStrategy = JsonTypeReflector.GetContainerNamingStrategy(
                        attribute
                    )!;
// is now
                NamingStrategy namingStrategy = JsonTypeReflector
                    .GetContainerNamingStrategy(containerAttribute)
                    !;
// probably should be
                NamingStrategy namingStrategy = JsonTypeReflector
                    .GetContainerNamingStrategy(containerAttribute)!;

            IsUnion = JsonTypeReflector.ReflectionDelegateFactory.CreateMethodCall<object?>(
                isUnionMethodInfo
            )!;
// is now
            IsUnion = JsonTypeReflector.ReflectionDelegateFactory
                .CreateMethodCall<object?>(isUnionMethodInfo)
                !;

            MethodCall<object?, object> invoke =
                JsonTypeReflector.ReflectionDelegateFactory.CreateMethodCall<object?>(invokeFunc)!;
// is now
            MethodCall<object?, object> invoke = JsonTypeReflector.ReflectionDelegateFactory
                .CreateMethodCall<object?>(invokeFunc)
                !;

    private static readonly MethodInfo CallPropertySetterOpenGenericMethod =
        typeof(PropertySetter).GetMethod(
            nameof(CallPropertySetter),
            BindingFlags.NonPublic | BindingFlags.Static
        )!;
// is now
    private static readonly MethodInfo CallPropertySetterOpenGenericMethod = typeof(PropertySetter)
        .GetMethod(nameof(CallPropertySetter), BindingFlags.NonPublic | BindingFlags.Static)
        !;


        var deserialized =
            (JSInProcessObjectReference)JsonSerializer.Deserialize<IJSInProcessObjectReference>(
                json,
                JsonSerializerOptions
            )!;
// is now
        var deserialized = (JSInProcessObjectReference)JsonSerializer
            .Deserialize<IJSInProcessObjectReference>(json, JsonSerializerOptions)
            !;

// is it weird if we do something like?
        _uiThreadId = (int)_window
            .GetType()
            .GetField("_managedThreadId", BindingFlags.NonPublic | BindingFlags.Instance)
            !.GetValue(_window)!;

[shocklateboy92]

Hmmmm. You know, when I first skimmed through the .cst files everything looked "fine".
We've probably been overthinking this. The vast majority of people won't look at this in detail and accept whatever we output without question.