Should code be refactored to more separate the Delivery, CM API and common parts?

kontent-ai / model-generator-net

Kontent.ai .NET model generator.

https://www.nuget.org/packages/Kontent.Ai.ModelGenerator

MIT License

17 stars 19 forks source link

Should code be refactored to more separate the Delivery, CM API and common parts? #53

Closed JanLenoch closed 6 years ago

JanLenoch commented 6 years ago

The current implementation is quite condensed in the ClassCodeGenerator .

What are your opinions on refactoring the code to more cleanly separate the above three parts? Is it worth it at the moment?

petrsvihlik commented 6 years ago

Nice to have. Could you pls be more specific about what should be extracted and how/where?

JanLenoch commented 6 years ago

Originally, this question was placed by @huluvu21 and @sutrkiller who have implemented the current version. I'll try to get the details from them and do an in-depth review eventually (if guys don't have more specific opinions about how to refactor).

JanLenoch commented 6 years ago

No requests have been made to do the refactoring, apart from the original one.