jorgecarleitao commented on a change in pull request #8380:
URL: https://github.com/apache/arrow/pull/8380#discussion_r501123505



##########
File path: .github/workflows/dev_labeler.yml
##########
@@ -16,8 +16,10 @@
 # under the License.
 
 name: PR labeler
-on:
-- pull_request_target
+on: pull_request_target
+  types: [opened, reopened]

Review comment:
       When a new push is made to the PR, the labels may change (as new code 
may change other parts of the code-base, or no longer change other parts). I 
think that this regresses on that feature.

##########
File path: .github/workflows/dev_labeler/labeler.yml
##########
@@ -16,7 +16,7 @@
 # under the License.
 
 lang-rust:
-  - rust/**/*
+  - rust/**

Review comment:
       I am not sure this will work. At least the [documentation 
requests](https://github.com/actions/labeler#basic-examples) `**/*`. If you 
tested and it still works, fine by me.




----------------------------------------------------------------
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:
[email protected]


Reply via email to