• Updated Unreleased section in CHANGELOG or it's not notable changes.
  • It's not a notable change.

Hi, I found closers that aren't closed in doghouse. Since it's minor fix, feel free to ignore this.

Go Doc comment on datastore.NewClient says:

// Call (*Client).Close() when done with the client.

Implementation of (*Client).Close() looks to close connection pooling.

// Close closes the Client. Call Close to clean up resources when done with the
// Client.
func (c *Client) Close() error {
	return c.connPool.Close()
}

So it looks better to be closed.

Adding context, I'm not familiar doghouse at all (though I'm interested in reviewdog). Even I don't know how to run doghouse (#1194). I'm developing a static code analyzer and randomly scanning open projects to find practical false positives so that I can improve recall of the analyzer. I just found this problem by the analyzer and consider it's a true positive.