guyuqi commented on pull request #12540:
URL: https://github.com/apache/arrow/pull/12540#issuecomment-1068703853


   > @guyuqi Can you add the line `RUN GO111MODULE=on go install 
honnef.co/go/tools/cmd/staticcheck@latest` to the 
`ci/docker/debian-10-go.dockerfile` and `ci/docker/debian-11-go.dockerfile` and 
update the go scripts to just reference `staticcheck` directly?
   > 
   > For the Windows CGO situation, i think the issue is that it is pulling in 
go version 1.17 and we need to run that `go install` command without the manual 
`GO111MODULE=on` (that need changed with go1.17). So you can add:
   > 
   > ```
   >       - name: Install staticcheck
   >         run: go install honnef.co/go/tools/cmd/staticcheck@latest
   > ```
   > 
   > to `.github/workflows/go.yml` at line 262 (before running go_build.sh for 
the windows-mingw workflow)
   
   Sure.
   That's informative. I'll update the related files later, thanks.
   


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: [email protected]

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to