group all stdlib imports together

[ewgRa]

import (
	"os"
	"strconv"

	"path"

must be

import (
	"os"
	"strconv"
	"path"

Inpired by elastic/beats#6641 (comment)

[ewgRa]

closed by #8

[quasilyte]

Doesn't goimports already do that?

[ewgRa]

@quasilyte and yes and no

package main

import (
    "os"
)

import (
    "fmt"
)

import "strings"

func main() {
	fmt.Println(strings.Split("a", "aa"), os.Readlink("b"))
}

Will be not changed by goimports. Idea is that such code must looks like:

package main

import (
    "fmt"
    "os"
    "strings"
)

func main() {
	fmt.Println(strings.Split("a", "aa"), os.Readlink("b"))
}

[quasilyte]

Interesting, thanks. Maybe there is an open issue on goimports already or we can make one.

[ewgRa]

@quasilyte I will later check goimports issues, I think it is kind of expected behavior

[ewgRa]

@quasilyte can't find any already created similar issue, create one golang/go#26795, will see is it a feature or a bug

[quasilyte]

@ewgRa, nice! Thanks for doing it!