pachamaltese commented on pull request #10232:
URL: https://github.com/apache/arrow/pull/10232#issuecomment-831573973


   > The changes to tasks.yml looks good, however a few notes:
   > 
   >     * Could you rebase/squash your commits here so that we don't have all 
of that extraneous history? It should be fine when we squash + merge it, but we 
want to make sure we're being clean with this and it's not touching other 
things accidentally / unnecessarily
   > 
   >     * Dig more into the failure that you noted. What failed? Does it make 
sense that these changes are touching that?
   > 
   >     * Could you update the format of the PR to match our style (pick 
either of the two tickets we have for this to use as the ticket number + mark 
the other as a duplicate)
   
   sure !!
   
   no problem, I'll do that now and push again


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

For queries about this service, please contact Infrastructure at:
[email protected]


Reply via email to