GenerateSerializer should be compatible with the required keyword.

[jamescarter-le]

Currently the compiler throws an error when trying to use the required keyword with GenerateSerializer

[GenerateSerializer]
public class Foo
{
    [Id(0)] public required string Bar{ get; set; }

Gives error:

Required member 'Foo.Bar' must be set in the object initializer or attribute constructor.

[ReubenBond]

We will need to fix this ASAP. Thank you for reporting it, @jamescarter-le.
I think the only viable fix is to avoid using new for types which have required properties, even if they have an accessible zero-parameter constructor.

For now, as a workaround, I can think of a couple of options:

• Do not use required
• Add the [UseActivator] attribute to your types which have required properties.
• Avoid having an accessible (public or internal) zero-parameter constructor

[jamescarter-le]

I just dropped required for now as the least friction option. Thank you.

…

On Tue, Apr 4, 2023, 16:28 Reuben Bond ***@***.***> wrote: We will need to fix this ASAP. Thank you for reporting it, @jamescarter-le <https://github.com/jamescarter-le>. I think the only viable fix is to avoid using new for types which have required properties, even if they have an accessible zero-parameter constructor. For now, as a workaround, I can think of a couple of options: - Do not use required - Add the [UseActivator] attribute to your types which have required properties. - Avoid having an accessible (public or internal) zero-parameter constructor — Reply to this email directly, view it on GitHub <#8367 (comment)>, or unsubscribe <https://github.com/notifications/unsubscribe-auth/AAMRJLCIDXKCSSKDFFDGFW3W7Q4Y7ANCNFSM6AAAAAAWSU2VNA> . You are receiving this because you were mentioned.Message ID: ***@***.***>

[onionhammer]

Any update on this?

[ReubenBond]

I am looking into this now. For types with required members, we will need to not call the constructor, since the error cannot be suppressed and it is emitted at the constructor call site.

[ReubenBond]

Fix PR opened: #8595. Thank you for bumping the issue