You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@arrow.apache.org by GitBox <gi...@apache.org> on 2022/11/30 21:15:57 UTC

[GitHub] [arrow] kou opened a new issue, #14795: PR title check linter checkes "[COMPONENT]" existence

kou opened a new issue, #14795:
URL: https://github.com/apache/arrow/issues/14795

   ### Describe the enhancement requested
   
   The current PR title check linter checks only "GH-XXX"/"ARROW-XXX"/"PARQUET-XXX".
   
   We want to check "[COMPONENT]" too.
   
   ### Component(s)
   
   Developer Tools


-- 
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: github-unsubscribe@arrow.apache.org.apache.org

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

[GitHub] [arrow] assignUser commented on issue #14795: [Dev] PR title check linter checkes "[COMPONENT]" existence

Posted by GitBox <gi...@apache.org>.
assignUser commented on issue #14795:
URL: https://github.com/apache/arrow/issues/14795#issuecomment-1339544825

   We could add all the format/meta-info checks as a separate job and make that obligatory so that it blocks merging until it passes.: https://cwiki.apache.org/confluence/display/INFRA/Git+-+.asf.yaml+features#Git.asf.yamlfeatures-Branchprotection


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] kou commented on issue #14795: [Dev] PR title check linter checkes "[COMPONENT]" existence

Posted by GitBox <gi...@apache.org>.
kou commented on issue #14795:
URL: https://github.com/apache/arrow/issues/14795#issuecomment-1332749257

   We may want to add this check to our lint CI: https://github.com/apache/arrow/blob/master/.github/workflows/dev.yml#L34


-- 
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: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow] kou commented on issue #14795: [Dev] PR title check linter checkes "[COMPONENT]" existence

Posted by GitBox <gi...@apache.org>.
kou commented on issue #14795:
URL: https://github.com/apache/arrow/issues/14795#issuecomment-1340028283

   Good!
   If we can implement all checks in `dev/merge_arrow_pr.py` with it, we may be able to use GitHub's "merge" button instead of `dev/merge_arrow_pr.py`.


-- 
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: github-unsubscribe@arrow.apache.org

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