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

chore(services): use AppName instead ModuleName for scaffold a new App #4010

Merged
merged 10 commits into from
Mar 19, 2024

Conversation

Pantani
Copy link
Collaborator

@Pantani Pantani commented Mar 12, 2024

Description

  • Use the var AppName instead of ModuleName in the templates;
  • Use the AppName to scaffold the chain app for the integration tests;

@Pantani Pantani self-assigned this Mar 12, 2024
@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs type:services Service-related issues. labels Mar 12, 2024
@Pantani Pantani changed the title feat: use AppName instead ModuleName for scaffold a new App feat(services): use AppName instead ModuleName for scaffold a new App Mar 12, 2024
@Pantani Pantani changed the title feat(services): use AppName instead ModuleName for scaffold a new App chore(services): use AppName instead ModuleName for scaffold a new App Mar 12, 2024
@julienrbrt julienrbrt added the backport/v28.x.y Backport to v28.x.y label Mar 12, 2024
julienrbrt
julienrbrt previously approved these changes Mar 12, 2024
jeronimoalbi
jeronimoalbi previously approved these changes Mar 12, 2024
@Pantani Pantani dismissed stale reviews from julienrbrt and jeronimoalbi via 6078d5d March 14, 2024 02:43
@julienrbrt julienrbrt enabled auto-merge (squash) March 18, 2024 15:17
@julienrbrt julienrbrt disabled auto-merge March 18, 2024 20:10
@julienrbrt
Copy link
Member

Looks like there an issue with my IBC integration tests. After all those merge of the main branch I still haven't them seen pass once.

@julienrbrt julienrbrt enabled auto-merge (squash) March 19, 2024 19:36
Copy link

codecov bot commented Mar 19, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 26.06%. Comparing base (6ec0274) to head (c011bbb).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             main    #4010   +/-   ##
=======================================
  Coverage   26.06%   26.06%           
=======================================
  Files         304      304           
  Lines       25440    25440           
=======================================
  Hits         6632     6632           
  Misses      18232    18232           
  Partials      576      576           
Files Coverage Δ
ignite/services/plugin/scaffold.go 56.81% <100.00%> (ø)

@julienrbrt julienrbrt merged commit 276b515 into main Mar 19, 2024
46 checks passed
@julienrbrt julienrbrt deleted the feat/use-app-name branch March 19, 2024 21:26
mergify bot pushed a commit that referenced this pull request Mar 19, 2024
…w App (#4010)

* use AppName instead ModuleName for scaffold a new App

* add changelog

* fix the quote for the scaffold chain for integration tests

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 276b515)
julienrbrt added a commit that referenced this pull request Mar 19, 2024
…w App (backport #4010) (#4040)

* chore(services): use `AppName` instead `ModuleName` for scaffold a new App (#4010)

* use AppName instead ModuleName for scaffold a new App

* add changelog

* fix the quote for the scaffold chain for integration tests

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
(cherry picked from commit 276b515)

* fix changelog

---------

Co-authored-by: Danilo Pantani <danpantani@gmail.com>
Co-authored-by: Julien Robert <julien@rbrt.fr>
julienrbrt added a commit that referenced this pull request May 29, 2024
…w App (#4010)

* use AppName instead ModuleName for scaffold a new App

* add changelog

* fix the quote for the scaffold chain for integration tests

---------

Co-authored-by: Pantani <Pantani>
Co-authored-by: Julien Robert <julien@rbrt.fr>
Co-authored-by: mergify[bot] <37929162+mergify[bot]@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backport/v28.x.y Backport to v28.x.y component:ci CI/CD workflow and automated jobs. component:configs type:services Service-related issues.
Projects
Status: Review / QA
Development

Successfully merging this pull request may close these issues.

3 participants