Closed thockin closed 11 months ago
LGTM and avoids the "goto" statement that I proposed - thanks.
Not sure why we're using channels here at all TBH, but .. lgtm
/approve /lgtm
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: justinsb, thockin
The full list of commands accepted by this bot can be found here.
The pull request process is described here
The channel is just a simple way of consuming an input string. We could have saved an index, just as easily.
Alternative to #785
k/k uses goleak, too, so it seemed a reasonable dep.