go-imap
go-imap copied to clipboard
Code should use unidirectional channels, where appropriate
The API docs would be easier to read if e.g. Client.Fetch took a write-only channel. Then it would be clear that it's an output.
Indeed. Unfortunately this doesn't bode well with our (arguably bad) implementation in responses.Fetch, which is used for both the client and the server.
Closing because go-imap v2 stops using Go channels in the public API.