GoImport broken when adding to empty imports.

[icholy]

What did you do?

package main

import ()

Execute: :GoImport time

What did you expect to happen?

package main

import (
	"time"
)

What happened instead?

package main

import ()
	"time"

Configuration

• vim version: 8.0 Included patches: 1-1207
• vim-go version: 0878992 (HEAD)
• go version (go version): 1.9
• operating system: Ubuntu 16.04
• vimrc you used to reproduce:

set nocompatible

" Plugins
call plug#begin('~/.vim/plugged')

Plug 'fatih/vim-go'

call plug#end()

[bhcleek]

Thank you for taking the time to report this. It looks to be a bug in goimports, though.

[icholy]

@bhcleek :GoImport doesn't use goimports

[bhcleek]

You're right, of course.

[fatih]

This is a very rare use case in my opinion. Not sure why one would want to write an empty import statement and then later import something? Or do I miss something here?

I know it's not related, but tools like goimports even don't allow to save it in this format, they remove the empty import () statement automatically.

[fatih]

This if fixed now in master, will be part of the upcoming release.