[GitHub] [arrow] dragosmg commented on pull request #13620: ARROW-17084: [R] Install the package before linting

[GitBox <gi...@apache.org>]

dragosmg commented on PR #13620:
URL: https://github.com/apache/arrow/pull/13620#issuecomment-1202301926

   @jonkeane Thanks for your review. I'll try to answer the questions below:
   * we do not currently have a CI job to test this. It's also a bit trickier to reproduce 
   * I noticed the failure (false-positive) during work on PR #13160 when the CI was picking up a lint in a file that did not exist in my branch - the CI run can still be seen [here](https://github.com/dragosmg/arrow/runs/7358965910?check_suite_focus=true), although the history of the branch was re-written due to a rebase. At the moment of the CI run, `test-dplyr-glimpse.R` did not exist in my branch, but it was in master.
   * another odd aspect (a false negative) is that the lint in `test-dplyr-glimpse.R` was not picked up as part of the PR in which it got added.  


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