Try to give a better name for migration step

davidsvantesson · davidsvantesson · commit b488f1d98c8d · 2019-11-17T17:04:37.000+01:00
diff --git a/models/migrations/migrations.go b/models/migrations/migrations.go
@@ -273,7 +273,7 @@ var migrations = []Migration{
 	// v108 -> v109
 	NewMigration("Add comment_id on table notification", addCommentIDOnNotification),
 	// v109 -> v110
-	NewMigration("update branch protection whitelist enable", addBranchProtectionAddEnableWhitelist),
+	NewMigration("update branch protection for can push and whitelist enable", addBranchProtectionCanPushAndEnableWhitelist),
 }
 
 // Migrate database to current version
diff --git a/models/migrations/v109.go b/models/migrations/v109.go
@@ -8,7 +8,7 @@ import (
 	"xorm.io/xorm"
 )
 
-func addBranchProtectionAddEnableWhitelist(x *xorm.Engine) error {
+func addBranchProtectionCanPushAndEnableWhitelist(x *xorm.Engine) error {
 
 	type ProtectedBranch struct {
 		CanPush                  bool  `xorm:"NOT NULL DEFAULT false"`

Original file line number	Diff line number	Diff line change
`@@ -273,7 +273,7 @@ var migrations = []Migration{`
`273`	`273`	`// v108 -> v109`
`274`	`274`	`NewMigration("Add comment_id on table notification", addCommentIDOnNotification),`
`275`	`275`	`// v109 -> v110`
`276`		`- NewMigration("update branch protection whitelist enable", addBranchProtectionAddEnableWhitelist),`
	`276`	`+ NewMigration("update branch protection for can push and whitelist enable", addBranchProtectionCanPushAndEnableWhitelist),`
`277`	`277`	`}`
`278`	`278`
`279`	`279`	`// Migrate database to current version`
Original file line number	Diff line number	Diff line change
`@@ -8,7 +8,7 @@ import (`
`8`	`8`	`"xorm.io/xorm"`
`9`	`9`	`)`
`10`	`10`
`11`		`-func addBranchProtectionAddEnableWhitelist(x *xorm.Engine) error {`
	`11`	`+func addBranchProtectionCanPushAndEnableWhitelist(x *xorm.Engine) error {`
`12`	`12`
`13`	`13`	`type ProtectedBranch struct {`
`14`	`14`	CanPush bool `xorm:"NOT NULL DEFAULT false"`