You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@orc.apache.org by GitBox <gi...@apache.org> on 2021/04/20 09:36:16 UTC

[GitHub] [orc] pgaref opened a new pull request #689: ORC-787: Update site to use `main`

pgaref opened a new pull request #689:
URL: https://github.com/apache/orc/pull/689


   ### What changes were proposed in this pull request?
   Update site to use `main`
   
   
   ### Why are the changes needed?
   Change needed after INFRA-21723
   
   
   ### How was this patch tested?
   Locally build and run site
   


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

[GitHub] [orc] pgaref commented on a change in pull request #689: ORC-787: Update site to use `main`

Posted by GitBox <gi...@apache.org>.
pgaref commented on a change in pull request #689:
URL: https://github.com/apache/orc/pull/689#discussion_r617439507



##########
File path: site/README.md
##########
@@ -7,7 +7,7 @@ the site is to use docker to use a standard environment.
 ## Setup
 
 1. `cd site`
-2. `git clone git@github.com:apache/orc.git -b asf-site target`

Review comment:
       Sure, updated




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

[GitHub] [orc] dongjoon-hyun merged pull request #689: ORC-787: Update site to use `main`

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun merged pull request #689:
URL: https://github.com/apache/orc/pull/689


   


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

[GitHub] [orc] dongjoon-hyun commented on pull request #689: ORC-787: Update site to use `main`

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on pull request #689:
URL: https://github.com/apache/orc/pull/689#issuecomment-824509747


   Since all comments are addressed, I'll merge this. Thank you all!


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

[GitHub] [orc] dongjoon-hyun commented on pull request #689: ORC-787: Update site to use `main`

Posted by GitBox <gi...@apache.org>.
dongjoon-hyun commented on pull request #689:
URL: https://github.com/apache/orc/pull/689#issuecomment-824346919


   Do you have any other comments, @omalley ?


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

[GitHub] [orc] omalley commented on a change in pull request #689: ORC-787: Update site to use `main`

Posted by GitBox <gi...@apache.org>.
omalley commented on a change in pull request #689:
URL: https://github.com/apache/orc/pull/689#discussion_r617097859



##########
File path: site/README.md
##########
@@ -7,7 +7,7 @@ the site is to use docker to use a standard environment.
 ## Setup
 
 1. `cd site`
-2. `git clone git@github.com:apache/orc.git -b asf-site target`

Review comment:
       Since this is for developers, I'd think that the ssh version is better.




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