-
Notifications
You must be signed in to change notification settings - Fork 54
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Code generation for large screen preview #169
Comments
Hi, I'm here to take it 🤚 |
Hey @mhmd-android, |
Hey @Kaaveh |
Great, thanks @mhmd-android |
Hi dear @Kaaveh After a thorough review of the proposed solution to use code generation to address the current issue in the ComposeNews project, I've concluded that this approach might not be the best solution for several reasons: Limited Utility: While the idea of using a code generator for this specific problem might seem appealing, the truth is that its application would be limited to this particular case only, offering no reusability in other parts of the project or future projects. Unnecessary Complexity: Introducing a code generation layer into the project could add unwanted complexity. This complexity might complicate code maintenance and understanding for other developers, especially those new to the project. Focus on More Pragmatic Solutions: It would be more beneficial to explore solutions that enhance the code's maintainability and readability without introducing additional layers that might not add significant value. Thank you for considering my perspective on this matter. |
@mhmd-android No problem, I think we can close this issue 👍 |
According to our conversion in #166
The text was updated successfully, but these errors were encountered: