migrate icon indicating copy to clipboard operation
migrate copied to clipboard

Change GracefulStop into a RO channel

Open Freeaqingme opened this issue 6 years ago • 1 comments

Migrate never writes to the GracefulStop channel, so why not guarantee that by turning the chan into a ReadOnly channel?

In my case I already had such a channel available which happened to be RO, so for me it wasn't just a nice to have, but sort of a necessity :)

I believe this change does not break BC.

Freeaqingme avatar Sep 22 '17 00:09 Freeaqingme

Apparently the CLI tooling does write directly to the chan. Coming to think of it, other people may do that as well, since it's an exported property. It may be a nice change for the next major version then (given that I also fix the test)?

Freeaqingme avatar Sep 22 '17 00:09 Freeaqingme