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

Posted to github@arrow.apache.org by "yevgenypats (via GitHub)" <gi...@apache.org> on 2023/04/12 16:36:18 UTC

[GitHub] [arrow] yevgenypats commented on pull request #34986: GH-34657: [Go] Add ValueString(i int) string to array

yevgenypats commented on PR #34986:
URL: https://github.com/apache/arrow/pull/34986#issuecomment-1505586519

   Thanks @zeroshade ! I addressed other comments. I only have one outstanding question for the template indentation and yeah if your or someone would be able to set the CI step that that does that automatically would be awesome as Im not super familiar with the arrow CI (so someone knowledgable would be able to do that faster I think - Opened issue - https://github.com/apache/arrow/issues/35079 )


-- 
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