`[NotClientImplementable]` does not prevent builds about avalonia

[maxkatz6]

That's mostly by design, as we wanted to keep non-ref assemblies with no changes. Making it possible to access private APIs if needed (for example, if somebody writes a custom backend).

But that's not ideal situation. Yes.

Alternative idea was to use C# analyzers, which would work better in this scenario. But won't support F# or other non-C# languages.

[robloo]

Personally, I think it's best to allow this usage. It's a good middle road: you get the warning telling you not to use the API. If you need it or know better you can override.

[colejohnson66]

I would be fine with that as well, but I'd expect an analyzer warning I can silence. With the current setup, Rider shows an error that I can't get rid of.

[maxkatz6]

With the current setup, Rider shows an error

As it should.

that I can't get rid of.

<AvaloniaAccessUnstablePrivateApis>true</AvaloniaAccessUnstablePrivateApis>

[colejohnson66]

I think my gripe is that it's an error, not a warning. Yet, it still behaves like a warning. I'm fine with Rider flagging usage, but not in a way that abuses ref assemblies.

For example, for #8266, I was told to try reimplementing LinearGradientBrush myself with some changes to test a potential fix. However, the error in Rider led me to believe that I couldn't, so I didn't even try. If it was instead a warning about an "unstable" interface, I could've just ignored it to try.

Also, that property is not documented anywhere on the website.

[maxkatz6]

Also, that property is not documented anywhere on the website.

Correct. We don't want users to rely on this property, as these APIs are unstable and can break in minor releases. Still, this option exists.

[kekekeks]

This is rather interesting. Previously Roslyn was using reference assemblies for compilation and the build would fail.
Perhaps something got changed in .NET SDK?

[kekekeks]

Actually, CoreCompile is being fed with 11.0.0/ref/net6.0/Avalonia.dll, not sure why compiler ignores the internal interface member there.