search

MarimerLLC / csla

A home for your business logic in any .NET application.
https://cslanet.com
MIT License
1.27k stars 406 forks source link

Rename Csla.Generators to Csla.Generator.AutoGenerator #4085

Closed rockfordlhotka closed 4 months ago

rockfordlhotka commented 4 months ago

Now that the Csla.Generators/cs project works (#3068), I think we need to reconsider how it is named - including the resulting NuGet package.

We may create other generators over time, some that might not have to do with serialization. Difficult to see. Always in motion is the future.

To avoid confusion, I think the generator we now have should be renamed to Csla.Generator.AutoSerializer so it corresponds to the attribute an end developer uses to invoke it. That'll tie the behavior and names together - for our codebase and also for the NuGet package name.

kcabral817 commented 4 months ago

Rocky,

Are you channeling a little Yoda? “Difficult to see. Always in motion is the future.”

Have a nice day.

Kevin

From: Rockford Lhotka @.> Sent: Sunday, July 7, 2024 5:39 PM To: MarimerLLC/csla @.> Cc: Subscribed @.***> Subject: [MarimerLLC/csla] Rename Csla.Generators to Csla.AutoGenerator (Issue #4085)

Now that the Csla.Generators/cs project works, I think we need to reconsider how it is named - including the resulting NuGet package.

We may create other generators over time, some that might not have to do with serialization. Difficult to see. Always in motion is the future.

To avoid confusion, I think the generator we now have should be renamed to Csla.AutoSerializer so it corresponds to the attribute an end developer uses to invoke it. That'll tie the behavior and names together - for our codebase and also for the NuGet package name.

— Reply to this email directly, view it on GitHubhttps://github.com/MarimerLLC/csla/issues/4085, or unsubscribehttps://github.com/notifications/unsubscribe-auth/AGJTVDVIL26NJFHXJAXZRRTZLHGTHAVCNFSM6AAAAABKPYPJRWVHI2DSMVQWIX3LMV43ASLTON2WKOZSGM4TIMRXGQ3TENY. You are receiving this because you are subscribed to this thread.Message ID: @.***>

StefanOssendorf commented 4 months ago

I think we should include clearly in the name that the package is a source generator. Maybe something like Csla.Generator.<GenFunctionalty> for example Csla.Generator.AutoSerialization?

luizfbicalho commented 4 months ago

Did anyone get this to do?

rockfordlhotka commented 4 months ago

No, would you like to do it?