Contributor

I applied a few of the modernize rules to simplify the code slightly:

• slices.Contains removes some chunks of iterations that do exactly what can be leveraged to slices.Contains
• for range N which is syntactic sugar.
• A single // +build which definitely needs to go.
• slices.Sort instead of sort.Slice.

The golangci-lint version is a bit old, and I didn't want to tamper around with it in this PR.

Contributor Author

@s1061123 Care to have a look? I could enable the modernize linter with some rules in golangci-lint.yaml

Contributor

I don't have so much time these days, hence I recommend to find another guy to review...

Contributor

In addition, this PR and your another one seems to be 'golang syntax improvement to modern', hence the priority looks lower than other serious one (such as bug fix and security one and so on).
Currently unfortunately we need a lot of such tasks, so please wait for a while.