forked from wrenn/wrenn
Fix lint warnings: drop deprecated Name field from snapshot response, check errcheck in benchmark
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
@ -129,7 +129,6 @@ func (s *Server) CreateSnapshot(
|
|||||||
return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("create snapshot: %w", err))
|
return nil, connect.NewError(connect.CodeInternal, fmt.Errorf("create snapshot: %w", err))
|
||||||
}
|
}
|
||||||
return connect.NewResponse(&pb.CreateSnapshotResponse{
|
return connect.NewResponse(&pb.CreateSnapshotResponse{
|
||||||
Name: msg.Name,
|
|
||||||
SizeBytes: sizeBytes,
|
SizeBytes: sizeBytes,
|
||||||
}), nil
|
}), nil
|
||||||
}
|
}
|
||||||
|
|||||||
@ -113,6 +113,6 @@ func BenchmarkParseSandboxID(b *testing.B) {
|
|||||||
s := FormatSandboxID(id)
|
s := FormatSandboxID(id)
|
||||||
b.ResetTimer()
|
b.ResetTimer()
|
||||||
for i := 0; i < b.N; i++ {
|
for i := 0; i < b.N; i++ {
|
||||||
ParseSandboxID(s)
|
_, _ = ParseSandboxID(s)
|
||||||
}
|
}
|
||||||
}
|
}
|
||||||
|
|||||||
Reference in New Issue
Block a user