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

[WIP] feat(services): scaffold api #4130

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from
Draft

Conversation

julienrbrt
Copy link
Member

@julienrbrt julienrbrt commented May 10, 2024

DO NOT REVIEW

ref: #3051

The scaffold API consists now of the following functions:

  • AddModule(ctx context.Context, m *Module) error
    To be used for recent modules that support depinject, it wires the module in app_config.go and app.go
    It auto-detects the folder architecture of the module via code analysis.

  • AddNonDepinjectModule(ctx context.Context, m *Module) error
    This does the same as AddModule expects it adds registers the module manually (by modifying RegisterStores and RegisterModules on the runtime app)

  • AddCommand(ctx context.Context, c *Command) error
    AddCommand simply add custom commands in the commands.go file of an app (of where the root command is registered).

That service focuses on the scaffolding of an app (and not modules). It can typically be used to replace logic from the wasm app and as basis of the future Cosmod app.

@github-actions github-actions bot added component:ci CI/CD workflow and automated jobs. component:configs type:services Service-related issues. component:packages labels May 10, 2024
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:packages type:services Service-related issues.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant