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 2021/12/07 07:36:00 UTC

[GitHub] [arrow] AlenkaF edited a comment on pull request #11873: ARROW-14242: Exposing the correct `indent` paramenter in `to_string`

AlenkaF edited a comment on pull request #11873:
URL: https://github.com/apache/arrow/pull/11873#issuecomment-987647700


   Thank you Marlene!
   The PR looks good. I do have some suggestions for improvements.
   
   The CI is giving errors for tests using `to_string()` in `test_array.py`. I suggest to run the tests locally to see what has happened due to the change in the code and make the necessary corrections.
   
   You mentioned you will add a test also, you could then add `indent` and `top_level_indent` specified to be different than 0.


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