Skip to content
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

Simplify scaffolded Ignite App to the bare minimum #3828

Closed
jeronimoalbi opened this issue Dec 7, 2023 · 0 comments · Fixed by #3839
Closed

Simplify scaffolded Ignite App to the bare minimum #3828

jeronimoalbi opened this issue Dec 7, 2023 · 0 comments · Fixed by #3839
Assignees
Labels
component:apps Related to Ignite Apps. component:code-gen Related to code generation

Comments

@jeronimoalbi
Copy link
Member

jeronimoalbi commented Dec 7, 2023

Ignite Apps can be scaffolded using ignite app scaffold but the resulting code have comments and examples that in most of the cases would end up being removed. This can be seen in the example app.

Scaffolding a minimum app should be the default. Ignite Apps are not complex and can be learned by reading the official Ignite documentation on how to write Apps.

The current efforts on having a repo with different basic examples of Ignite Apps also makes scaffolding extra code and comments unnecessary.

It would also be entirely possible to add simple comments to the scaffolded apps where a link to the examples repository could be added.

@jeronimoalbi jeronimoalbi added component:apps Related to Ignite Apps. component:code-gen Related to code generation labels Dec 7, 2023
@Ehsan-saradar Ehsan-saradar self-assigned this Dec 7, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:apps Related to Ignite Apps. component:code-gen Related to code generation
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants