From 98be3cbf6fd3d12a0abc266ec0aef903ba74f103 Mon Sep 17 00:00:00 2001 From: Sorawis Nilparuk Date: Wed, 9 Jul 2025 09:59:59 -0700 Subject: [PATCH] use named return for retErr --- internal/db/schema/manager.go | 7 ++----- 1 file changed, 2 insertions(+), 5 deletions(-) diff --git a/internal/db/schema/manager.go b/internal/db/schema/manager.go index 9f2426085a..000c8aca93 100644 --- a/internal/db/schema/manager.go +++ b/internal/db/schema/manager.go @@ -243,12 +243,9 @@ func (b *Manager) ApplyMigrations(ctx context.Context) ([]RepairLog, error) { // runMigrations passes migration queries to a database driver and manages // the version and dirty bit. Cancellation or deadline/timeout is managed // through the passed in context. -func (b *Manager) runMigrations(ctx context.Context, p *provider.Provider) ([]RepairLog, error) { +func (b *Manager) runMigrations(ctx context.Context, p *provider.Provider) (logEntries []RepairLog, retErr error) { const op = "schema.(Manager).runMigrations" - var logEntries []RepairLog - var retErr error - if startErr := b.driver.StartRun(ctx); startErr != nil { return nil, errors.Wrap(ctx, startErr, op) } @@ -312,5 +309,5 @@ func (b *Manager) runMigrations(ctx context.Context, p *provider.Provider) ([]Re return nil, errors.Wrap(ctx, err, op) } - return logEntries, retErr + return logEntries, nil }