Bump Go to v1.23.7 (#30455)

* Update Go version to v1.23.7

* Bump golangci-lint to a version supporting Go 1.23

* Fix golangci-lint warnings

Several rules from gosimple, revive and staticcheck linters were
failing:
- Redefinition of built-in identifiers (max, min, new, recover...)
- Use of printf-like functions with simple strings
- Check for nil slices, when len already takes it into account

* Trigger Build

* Trigger Build

---------

Co-authored-by: Mattermost Build <build@mattermost.com>

server/channels/store/sqlstore/migrate.go

@@ -180,13 +180,13 @@ func (ss *SqlStore) migrate(direction migrationDirection, dryRun bool) error {
 	}
 }
 
-func (m *Migrator) GeneratePlan(recover bool) (*models.Plan, error) {
+func (m *Migrator) GeneratePlan(shouldRecover bool) (*models.Plan, error) {
 	diff, err := m.engine.Diff(models.Up)
 	if err != nil {
 		return nil, err
 	}
 
-	plan, err := m.engine.GeneratePlan(diff, recover)
+	plan, err := m.engine.GeneratePlan(diff, shouldRecover)
 	if err != nil {
 		return nil, err
 	}