refactor: fix linter

evgeniy-scherbina · evgeniy-scherbina · commit 087bd209f854 · 2025-04-23T19:16:42.000Z
diff --git a/coderd/prebuilds/noop.go b/coderd/prebuilds/noop.go
@@ -38,12 +38,12 @@ var _ ReconciliationOrchestrator = NoopReconciler{}
 
 type AGPLPrebuildClaimer struct{}
 
-func (c AGPLPrebuildClaimer) Claim(context.Context, uuid.UUID, string, uuid.UUID) (*uuid.UUID, error) {
+func (AGPLPrebuildClaimer) Claim(context.Context, uuid.UUID, string, uuid.UUID) (*uuid.UUID, error) {
 	// Not entitled to claim prebuilds in AGPL version.
 	return nil, ErrNoClaimablePrebuiltWorkspaces
 }
 
-func (c AGPLPrebuildClaimer) Initiator() uuid.UUID {
+func (AGPLPrebuildClaimer) Initiator() uuid.UUID {
 	return uuid.Nil
 }
 
diff --git a/coderd/workspaces.go b/coderd/workspaces.go
@@ -886,8 +886,8 @@ func claimPrebuild(ctx context.Context, claimer prebuilds.Claimer, db database.S
 
 	lookup, err := db.GetWorkspaceByID(prebuildsCtx, *claimedID)
 	if err != nil {
-		logger.Error(ctx, "unable to find claimed workspace by ID", slog.Error(err), slog.F("claimed_prebuild_id", (*claimedID).String()))
-		return nil, xerrors.Errorf("find claimed workspace by ID %q: %w", (*claimedID).String(), err)
+		logger.Error(ctx, "unable to find claimed workspace by ID", slog.Error(err), slog.F("claimed_prebuild_id", claimedID.String()))
+		return nil, xerrors.Errorf("find claimed workspace by ID %q: %w", claimedID.String(), err)
 	}
 	return &lookup, nil
 }
diff --git a/enterprise/coderd/prebuilds/claim.go b/enterprise/coderd/prebuilds/claim.go
@@ -46,7 +46,7 @@ func (c EnterpriseClaimer) Claim(
 	return &result.ID, nil
 }
 
-func (_ EnterpriseClaimer) Initiator() uuid.UUID {
+func (EnterpriseClaimer) Initiator() uuid.UUID {
 	return prebuilds.SystemUserID
 }
 

Original file line number	Diff line number	Diff line change
`@@ -38,12 +38,12 @@ var _ ReconciliationOrchestrator = NoopReconciler{}`
`38`	`38`
`39`	`39`	`type AGPLPrebuildClaimer struct{}`
`40`	`40`
`41`		`-func (c AGPLPrebuildClaimer) Claim(context.Context, uuid.UUID, string, uuid.UUID) (*uuid.UUID, error) {`
	`41`	`+func (AGPLPrebuildClaimer) Claim(context.Context, uuid.UUID, string, uuid.UUID) (*uuid.UUID, error) {`
`42`	`42`	`// Not entitled to claim prebuilds in AGPL version.`
`43`	`43`	`return nil, ErrNoClaimablePrebuiltWorkspaces`
`44`	`44`	`}`
`45`	`45`
`46`		`-func (c AGPLPrebuildClaimer) Initiator() uuid.UUID {`
	`46`	`+func (AGPLPrebuildClaimer) Initiator() uuid.UUID {`
`47`	`47`	`return uuid.Nil`
`48`	`48`	`}`
`49`	`49`
Original file line number	Diff line number	Diff line change
`@@ -886,8 +886,8 @@ func claimPrebuild(ctx context.Context, claimer prebuilds.Claimer, db database.S`
`886`	`886`
`887`	`887`	`lookup, err := db.GetWorkspaceByID(prebuildsCtx, *claimedID)`
`888`	`888`	`if err != nil {`
`889`		`- logger.Error(ctx, "unable to find claimed workspace by ID", slog.Error(err), slog.F("claimed_prebuild_id", (*claimedID).String()))`
`890`		`- return nil, xerrors.Errorf("find claimed workspace by ID %q: %w", (*claimedID).String(), err)`
	`889`	`+ logger.Error(ctx, "unable to find claimed workspace by ID", slog.Error(err), slog.F("claimed_prebuild_id", claimedID.String()))`
	`890`	`+ return nil, xerrors.Errorf("find claimed workspace by ID %q: %w", claimedID.String(), err)`
`891`	`891`	`}`
`892`	`892`	`return &lookup, nil`
`893`	`893`	`}`
Original file line number	Diff line number	Diff line change
`@@ -46,7 +46,7 @@ func (c EnterpriseClaimer) Claim(`
`46`	`46`	`return &result.ID, nil`
`47`	`47`	`}`
`48`	`48`
`49`		`-func (_ EnterpriseClaimer) Initiator() uuid.UUID {`
	`49`	`+func (EnterpriseClaimer) Initiator() uuid.UUID {`
`50`	`50`	`return prebuilds.SystemUserID`
`51`	`51`	`}`
`52`	`52`