x/tools/gopls: the infertypeargs analyzer should be a "convenience" analyzer

[findleyr]

This analyzer does not report bugs, but rather potential refactorings. As such, it should be a "convenience" analyzer, which is less intrusive (or it should be disabled by default).

[xiaokentrl]

func main() {  
    f, err := os.Open("/test.txt") :not {
            fmt.Println(err)
    }


    //or
    
    f, err := os.Open("/test.txt") :not _

    //or
    f, err := os.Open("/test.txt") e: {
        fmt.Println(e)
    }

    f, err := os.Open("/test.txt") e:



}

[gopherbot]

Change https://go.dev/cl/472181 mentions this issue: gopls/internal/lsp/source: make infertypeargs a convenience analyzer