Azure / autorest.csharp

Extension for AutoRest (https://github.com/Azure/autorest) that generates C# code
MIT License
142 stars 165 forks source link

Clean up CSharpName helper methods #4767

Open live1206 opened 4 months ago

live1206 commented 4 months ago

Describe the issue or request Not we have CSharpName helper methods for parameter, property, operation and such on. We need to put them in a better place.

Describe your ideas for solutions It might be better to put them directly in parameter, property models.

m-nash commented 4 months ago

I think we should always have Name be the name we should use vs two name properties. If Name is needed as a unique identifier for lookups between input and output library we should have an Id field to represent this.

ShivangiReja commented 3 months ago

@live1206 - I don't see CSharpName helper methods in MGC, is this issue for autorest.csharp? If yes, can we remove MGC project for it?

live1206 commented 3 months ago

The helper methods only exists in autorest.csharp now, and we need to clean them up before moving to MGC.