The refactoring in #100 resulted in a function that takes a recurser ID (as int64) and a recurser struct (Recurser, which has the same ID as a field). We can simplify this by changing it to only need the struct parameter.
|
func (r *RecursersClient) Set(ctx context.Context, _ int64, recurser *Recurser) error { |
The name being an underscore acts as proof that the parameter is totally unused.
This might be a good time to try the new go fix inliner!
(Extracted from #100 (comment))
The refactoring in #100 resulted in a function that takes a recurser ID (as
int64) and a recurser struct (Recurser, which has the same ID as a field). We can simplify this by changing it to only need the struct parameter.pairing-bot/store/recursers.go
Line 114 in 235495c
The name being an underscore acts as proof that the parameter is totally unused.
This might be a good time to try the new
go fixinliner!(Extracted from #100 (comment))