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

feat: Improve app scaffolding #3839

Merged
merged 37 commits into from
Jan 29, 2024
Merged

Conversation

Ehsan-saradar
Copy link
Contributor

@Ehsan-saradar Ehsan-saradar commented Dec 13, 2023

Closes #3828

This PR:

  1. Removes unnecessary code generated with ignite app scaffold.
  2. Adds a cmd package to structure of app source code for cobra commands and sub commands.
  3. Adds an integration package for integration unit tests.
  4. Generates app.ignite.yml file at the root of project which will give some informations about the app(s) in the repository to the marketplace app.

@Ehsan-saradar Ehsan-saradar added component:scaffold Feature, enhancement, or refactor related to scaffolding. component:extensions Related to Ignite Extensions. labels Dec 13, 2023
@Ehsan-saradar Ehsan-saradar self-assigned this Dec 13, 2023
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs type:services Service-related issues. labels Dec 13, 2023
@Ehsan-saradar Ehsan-saradar force-pushed the feat/improve-app-scaffolding branch from 14af2d4 to 6dbf753 Compare December 13, 2023 11:05
@Ehsan-saradar Ehsan-saradar force-pushed the feat/improve-app-scaffolding branch from b466dbf to c3f3871 Compare December 13, 2023 22:04
Copy link

@Cplus13 Cplus13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Review complete

julienrbrt
julienrbrt previously approved these changes Jan 18, 2024
Copy link
Member

@julienrbrt julienrbrt left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm!

salmad3
salmad3 previously approved these changes Jan 20, 2024
@Ehsan-saradar Ehsan-saradar dismissed stale reviews from salmad3 and julienrbrt via 79f2474 January 20, 2024 12:44
julienrbrt
julienrbrt previously approved these changes Jan 22, 2024
@jeronimoalbi
Copy link
Member

There a a couple of comments to be resolved but other than that looks good 👍

Pantani
Pantani previously approved these changes Jan 26, 2024
@Pantani Pantani dismissed stale reviews from julienrbrt and themself via 99a08d5 January 29, 2024 20:59
@Pantani Pantani requested a review from salmad3 January 29, 2024 20:59
Pantani
Pantani previously approved these changes Jan 29, 2024
@Pantani Pantani requested a review from julienrbrt January 29, 2024 21:01
@Ehsan-saradar Ehsan-saradar merged commit 86ebb55 into main Jan 29, 2024
48 of 50 checks passed
@Ehsan-saradar Ehsan-saradar deleted the feat/improve-app-scaffolding branch January 29, 2024 21:57
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
component:ci CI/CD workflow and automated jobs. component:configs component:extensions Related to Ignite Extensions. component:packages component:scaffold Feature, enhancement, or refactor related to scaffolding. type:services Service-related issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Simplify scaffolded Ignite App to the bare minimum
6 participants