You are viewing a plain text version of this content. The canonical link for it is here.
Posted to dev@drill.apache.org by GitBox <gi...@apache.org> on 2021/03/09 21:51:12 UTC

[GitHub] [drill] fredster33 opened a new pull request #2188: Grammar edit

fredster33 opened a new pull request #2188:
URL: https://github.com/apache/drill/pull/2188


   # [DRILL-XXXX](https://issues.apache.org/jira/browse/DRILL-XXXX): Grammar Edit
   
   (this is a small grammar edit for the README, so I didn't file a JIRA issue)
   
   ## Description
   This PR fixed some grammar issues in the README.
   
   ## Documentation
   This updated the README.
   
   ## Testing
   None, this is a trivial change.


----------------------------------------------------------------
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] [drill] fredster33 commented on pull request #2188: [DOC UPDATE] Grammar edit

Posted by GitBox <gi...@apache.org>.
fredster33 commented on pull request #2188:
URL: https://github.com/apache/drill/pull/2188#issuecomment-794677329


   > @fredster33 Hello. We just need to add `[DOC UPDATE]` in front of the title of the PR when update the docs. Thanks for your contribution.
   
   Edited!


----------------------------------------------------------------
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] [drill] luocooong merged pull request #2188: [DOC UPDATE] Grammar edit

Posted by GitBox <gi...@apache.org>.
luocooong merged pull request #2188:
URL: https://github.com/apache/drill/pull/2188


   


----------------------------------------------------------------
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] [drill] dzamo commented on pull request #2188: [DOC UPDATE] Grammar edit

Posted by GitBox <gi...@apache.org>.
dzamo commented on pull request #2188:
URL: https://github.com/apache/drill/pull/2188#issuecomment-794836007


   @luocooong when the target branch is `gh-pages` I just go ahead and merge.  This commit is going to `master`, who should merge it?


----------------------------------------------------------------
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] [drill] luocooong commented on pull request #2188: [DOC UPDATE] Grammar edit

Posted by GitBox <gi...@apache.org>.
luocooong commented on pull request #2188:
URL: https://github.com/apache/drill/pull/2188#issuecomment-794847404


   @dzamo Thanks for the review. cgivre can help with the merge. me too


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