rf
rf copied to clipboard
refactor: keep blank imports
Previously blank imports were always removed which prevents using packages for init-only purposes. Updated tests to reflect this changes.
@rsc @mdempsky can you merge it? I have encountered this bug today and that commit fix it
Sorry for bumping up old issue, but this bug is indeed important, I'd be happy if it is fixed (by merging this PR or otherwise).
When blank imports are removed, the compiler won't complain, and so these incorrect changes are easy to miss.
@vearutop if you need a great replacement then try https://github.com/uber-go/gopatch