You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@pulsar.apache.org by GitBox <gi...@apache.org> on 2021/09/09 02:44:43 UTC

[GitHub] [pulsar] fengtao1998 opened a new pull request #11977: Add document label check robot

fengtao1998 opened a new pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977


   ### Motivation
   To improve doc workflow efficiency, we want to introduce Bot to Pulsar community
   
   ### Modifications
   
   1.Modified pull request template
   
   2.Added YML file for BOT running
   
   
   
   ### Documentation
   
   Check the box below and label this PR (if you have committer privilege).
   
   Need to update docs? 
   
   - [ ] `doc-required` 
     
     (If you need help on updating docs, create a doc issue)
     
   - [x] `no-need-doc` 
     
     (Please explain why)
     
   - [ ] `doc` 
     
     (If this PR contains doc changes)
   
   
   You can see the details of BOT here(https://docs.google.com/document/d/1bQfZkSu5nG1tNycpmXXtUFn-Z5-h-uqHv6IXsCEySQ8/edit#)
   
   You can try to test docbot in this repo(https://github.com/fengtao1998/cherry_test)
   
   If you need more BOT functions, please comment
   
   
   
   
   


-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r711668423



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       I have changed the repo in this GH Actions. 
   @eolivelli PTAL, thanks.




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r711668423



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       I have changed the repo in this GH Actions. 
   PTAL, thanks.




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] eolivelli commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
eolivelli commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r706005670



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       this is the repo
   https://github.com/apache/pulsar-test-infra
   
   having it in our repo is better, as private repos may disappear some day




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] eolivelli merged pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
eolivelli merged pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977


   


-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] eolivelli commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
eolivelli commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r705045468



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       I believe that is it not allowed in the ASF to use third party GH Actions using branch name references.
   Can we copy this code into an ASF repository ? or at least point to a specific SHA (if it is possible)




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] timmyyuan commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
timmyyuan commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r705066875



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       Can we put this action into https://github.com/apache/pulsar-test-infra ?




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r706008170



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       Thank you. I have submitted a pullrequest(https://github.com/apache/pulsar-test-infra/pull/21 ) in this repo. When the merge succeeds, I will apply for review again.




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] Anonymitaet commented on pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
Anonymitaet commented on pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#issuecomment-915719060


   @timmyyuan @codelipenghui could you please help review? Many thanks!


-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r706008170



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       Thank you. I have submitted a pullrequest in this repo. When the merge succeeds, I will apply for review again.




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r705970613



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       > I believe that is it not allowed in the ASF to use third party GH Actions using branch name references.
   > Can we copy this code into an ASF repository ? or at least point to a specific SHA (if it is possible)
   
   I have fixed the problem of GH action using third-party branches. I changed to point a sha to quote script. Do you think this is OK? Or can you tell me where the ASF repo is? I'll push the script in.
   




-- 
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: commits-unsubscribe@pulsar.apache.org

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



[GitHub] [pulsar] fengtao1998 commented on a change in pull request #11977: Add document label check robot

Posted by GitBox <gi...@apache.org>.
fengtao1998 commented on a change in pull request #11977:
URL: https://github.com/apache/pulsar/pull/11977#discussion_r706008170



##########
File path: .github/workflows/ci-documentbot.yml
##########
@@ -0,0 +1,27 @@
+name: Auto Labeling
+
+on:
+  pull_request:
+    types:
+      - opened
+      - edited
+      - labeled
+
+  
+
+# A GitHub token created for a PR coming from a fork doesn't have
+# 'admin' or 'write' permission (which is required to add labels)
+# To avoid this issue, you can use the `scheduled` event and run
+# this action on a certain interval.And check the label about the
+# document.
+
+jobs:
+  labeling:
+    runs-on: ubuntu-latest
+    steps:
+      - uses: actions/checkout@v2
+
+      - uses: pr-bot-test/auto-labeling@master

Review comment:
       Thank you. I have submitted a pullrequest(https://github.com/apache/pulsar-test-infra/pull/21) in this repo. When the merge succeeds, I will apply for review again.




-- 
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: commits-unsubscribe@pulsar.apache.org

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