Fixing CS7036 (generate constructor based on base class) should give an option to use Primary Constructor

[CyrusNajmabadi]

Discussed in #74598

^{Originally posted by datvm July 30, 2024}
I am not sure if this is a duplicate because I think this is a very popular scenario. Assume we have this code:

class BaseClass(CancellationToken ct)
{

}

class DerivedClass : BaseClass
{
}

VS would suggest to "Generate constructor 'DerivedClass(ct)'":

After that, VS continues to suggest to use Primary Constructor instead which gives this code:

class DerivedClass(CancellationToken ct) : BaseClass(ct)
{
}

I think the analyzer should merge the two steps into one?

[kinkeadian25]

@CyrusNajmabadi do you want it to offer to generate and be primary or not or to just generate and be primary?

[CyrusNajmabadi]

I think it would be ok to offer both "generate constructor" and "generate primary constructor"

[kinkeadian25]

Mind if I take this one? Finally have a machine that can run Roslyn again lol.

[CyrusNajmabadi]

Sure.