You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@sedona.apache.org by GitBox <gi...@apache.org> on 2021/08/06 18:58:20 UTC

[GitHub] [incubator-sedona] yitao-li opened a new pull request #535: use GITHUB_TOKEN to authenticate `remotes::install_github()` web request

yitao-li opened a new pull request #535:
URL: https://github.com/apache/incubator-sedona/pull/535


   Signed-off-by: Yitao Li <yi...@rstudio.com>
   
   ## Is this PR related to a proposed Issue?
   
   No.
   
   ## What changes were proposed in this PR?
   
   This is a trivial change to fix the GitHub rate-limiting problem encountered in https://github.com/apache/incubator-sedona/pull/534
   
   ## How was this patch tested?
   
   via GitHub Actions workflows for R builds
   
   ## Did this PR include necessary documentation updates?
   
   No.
   


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-sedona] jiayuasu merged pull request #535: use GITHUB_TOKEN to authenticate `remotes::install_github()` web requests

Posted by GitBox <gi...@apache.org>.
jiayuasu merged pull request #535:
URL: https://github.com/apache/incubator-sedona/pull/535


   


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-sedona] jiayuasu commented on pull request #535: use GITHUB_TOKEN to authenticate `remotes::install_github()` web requests

Posted by GitBox <gi...@apache.org>.
jiayuasu commented on pull request #535:
URL: https://github.com/apache/incubator-sedona/pull/535#issuecomment-894589250


   Thanks for your contribution!


-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org



[GitHub] [incubator-sedona] yitao-li commented on a change in pull request #535: use GITHUB_TOKEN to authenticate `remotes::install_github()` web requests

Posted by GitBox <gi...@apache.org>.
yitao-li commented on a change in pull request #535:
URL: https://github.com/apache/incubator-sedona/pull/535#discussion_r684469068



##########
File path: .github/workflows/r.yml
##########
@@ -40,6 +40,10 @@ jobs:
     env:
       SPARK_VERSION: ${{ matrix.spark }}
       SCALA_VERSION: ${{ matrix.scala }}
+      # Ensure the temporary auth token for this workflow, instead of the
+      # bundled GitHub PAT from the `remotes` package is used for
+      # `remotes::install_github()`
+      GITHUB_PAT: ${{ secrets.GITHUB_TOKEN }}

Review comment:
       OK. Looks like this definitely fixes all of the R package installation failure(s) from existing R builds.




-- 
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: dev-unsubscribe@sedona.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org