You are viewing a plain text version of this content. The canonical link for it is here.

Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/03/10 04:17:48 UTC

[GitHub] [arrow] guyuqi edited a comment on pull request #12540: ARROW-15296: [CI][GO] Add Go staticcheck linting to CI lint job

guyuqi edited a comment on pull request #12540:
URL: https://github.com/apache/arrow/pull/12540#issuecomment-1063647480


   > @guyuqi I really don't know why the Windows build is failing with that error unless it's somehow using an older version of Go than it should be. I think it might make sense to create a docker image to run staticcheck rather than manually running `go install` in the script. If you need i can provide a Dockerfile for it
   
   Thanks for your comments.
   Removed unused field rather than inserting the `U1000 `lint ignore in the latest commit.
   
   It makes sense to put `staticcheck ` installation into docker images.
   


-- 
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: github-unsubscribe@arrow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org