Exclude CS8618 for DbSet<T> in a derived DbContext class

[Youssef1313]

See this article from EF Core docs.

When nullable reference types are enabled, the C# compiler emits warnings for any uninitialized non-nullable property, as these would contain null. As a result, the common practice of defining a non-nullable DbSet on a context will now generate a warning. However, EF Core always initializes all DbSet properties on DbContext-derived types, so they are guaranteed to never be null, even if the compiler is unaware of this.

There is a workaround mentioned in the docs to add = null!, but I feel it's reasonable to exclude any public DbSet that exists in a class derived from DbContext by default.

[Dreamescaper]

I think that this issue doesn't belong here, should go to ef core repository.

As a workaround you can disable it for certain files via .editorconfig file.

[Youssef1313]

@Dreamescaper How EF Core's team may be able to fix that? I think it should be excluded by the compiler (less noisy than disabling warning or adding = null!).

[Youssef1313]

cc: @ajcvickers May you take a look?

[Dreamescaper]

They can create DiagnosticSuppressor for this purpose.

[Youssef1313]

@Dreamescaper Great. I didn't knew about it before, but it looks like the correct fix. I'll wait for @ajcvickers to move the issue to efcore repo