-
Notifications
You must be signed in to change notification settings - Fork 565
feat(templates): scaffold module migrations #3902
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
Conversation
|
||
// Migrations | ||
PlaceholderMigrationsRegister = "// this line is used by starport scaffolding # migrations/register" | ||
PlaceholderMigrationsMigrator = "// this line is used by starport scaffolding # migrations/migrator" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not particular to this update but generally, should the comments say "Ignite scaffolding"?
// MigrateStore migrates the module to consensus version {{ num }}. | ||
func MigrateStore(ctx context.Context, storeService corestoretypes.KVStoreService, cdc codec.BinaryCodec) { | ||
// Insert {{ num }} migration here. | ||
} |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
} | |
} | |
} | ||
} | ||
|
||
// this line is used by starport scaffolding # migrations/migrator |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
// this line is used by starport scaffolding # migrations/migrator | |
// this line is used by starport scaffolding # migrations/migrator | |
e3a997f
to
1da47a6
Compare
1da47a6
to
8e0937d
Compare
Reopening this. |
Closes: #3843