thisisnic edited a comment on pull request #12119: URL: https://github.com/apache/arrow/pull/12119#issuecomment-1010043935
> @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? I think it makes more sense here, though you could always mention it in both places if it fits in both? Maybe here could be the detailed version, and in the testing section you just briefly remind the reader they should recompile? Might be easier to have it as a separate PR - up to you! -- 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]
