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/01/11 14:11:30 UTC

[GitHub] [arrow] AlenkaF commented on pull request #12119: ARROW-14754: [Doc] Steps in making your first PR - building R package

AlenkaF commented on pull request #12119:
URL: https://github.com/apache/arrow/pull/12119#issuecomment-1010001310


   @thisisnic I think it is great! Good intro to the main part of the documentation.
   
   I did get an idea when looking at this PR to move notes about [recompiling cython, C++ ](https://github.com/apache/arrow/blob/master/docs/source/developers/guide/step_by_step/testing.rst) from testing section to this part of the guide. I think it fits better. And then a sentence or two could be added for the R package, that it needs no recompiling (if I understand correctly).
   
   What do you think? I could try to push a commit with the suggestion here?


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