AlenkaF commented on a change in pull request #11505:
URL: https://github.com/apache/arrow/pull/11505#discussion_r745370110



##########
File path: docs/source/developers/python.rst
##########
@@ -55,12 +55,15 @@ like so:
 
 .. code-block:: shell
 
-   pytest pyarrow
+   pytest arrow/python/pyarrow

Review comment:
       Yes, currently the instructions are written from the folder `repos`, 
which is above `arrow`. Even when building `pyarrow` you move from `repos` to 
`python` folder in `arrow` and than back to `repos` again.
   
   I am sure there are standard ways of working but for someone already dealing 
with build and scale of Arrow library I think this simple things should be 
written explicitly and without adding more confusion.
   
   What we can do is stay in python folder after building pyarrow (I would than 
change the instructions there) and leave pytest as it was?  What do u think?




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


Reply via email to