-
Notifications
You must be signed in to change notification settings - Fork 553
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
Remove placeholders from the template #2841
Comments
Hey @fadeev, if you don't remember, I was behind the project for addressing this during the 2021 Hackathon. I haven't really been keeping up to speed with the overall changes but I've been looking for a reason to finish this up and contribute back to the project. (Note: saw the discussion on the ignite-cli-dev channel on Discord but I guess it is restricted for team-only, lmk if there's a more appropriate place to discuss this) |
Oh, hey, @DimitrisJim! The |
Hi @DimitrisJim, have you made any progress on this ? We'd like to move forward so if you don't have the time to work on it, no problem but we need to assign someone else. |
@tbruyelle yes! Had some other issues pop up but I should hopefully get a PR within the week. |
awesome ! |
Bump for an FIY, I am working on the second part for this, albeit not at the pace I'd like. |
Currently, placeholders exist in the following places in the standard template:
app/app.go
proto/
genesis.proto
query.proto
tx.proto
x/MODULE/
client/cli
: the next version of the CLI will auto-generate CLI, so removing placeholders from here is not a prioritytypes/
codec.go
keys.go
genesis.go
andgenesis_test.go
genesis.go
andgenesis_test.go
module_simulation.go
cmd
: should be removed by replacingcosmoscmd
with something elseThis issue is related to the discussion on Discord:
https://discord.com/channels/893126937067802685/907904892780290058/1020618039647027290
The text was updated successfully, but these errors were encountered: