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/05/03 22:08:24 UTC

[GitHub] [arrow] jonkeane commented on pull request #10232: [CI] Remove test-conda-python-3.8-jpype build

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


   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) 


-- 
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:
users@infra.apache.org