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/04/17 10:05:35 UTC

[GitHub] [arrow-rs] tustvold opened a new issue, #1578: Require CI Checks

tustvold opened a new issue, #1578:
URL: https://github.com/apache/arrow-rs/issues/1578

   **Is your feature request related to a problem or challenge? Please describe what you are trying to do.**
   
   Currently prior to merge you need to check all the CI bar MIRI has run to completion. This is somewhat tiresome and potentially error prone.
   
   **Describe the solution you'd like**
   
   All CI checks bar MIRI should be required prior to merge. I don't know who has access to change this, perhaps @andygrove ??
   
   **Describe alternatives you've considered**
   
   We could not do this


-- 
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-rs] alamb commented on issue #1578: Require CI Checks

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #1578:
URL: https://github.com/apache/arrow-rs/issues/1578#issuecomment-1100871559

   I personally like the flexibility of not requiring CI checks (it allows me to accelerate things like README updates or if clippy has released a new breaking change).
   
   That being said, I am not opposed to this change; I just feel like it actually adds more burden on the maintainers.
   
   In terms of changing settings, I think apache projects don't control github settings directly, instead they check in a config file that then sets the permissions. 
   
   https://github.com/apache/arrow-rs/blob/master/.asf.yaml 


-- 
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-rs] tustvold commented on issue #1578: Require CI Checks

Posted by GitBox <gi...@apache.org>.
tustvold commented on issue #1578:
URL: https://github.com/apache/arrow-rs/issues/1578#issuecomment-1100944773

   If memory serves there is a way to override on a case-by-case basis, but potentially that require repos admin 🤔
   
   I'll close this as I don't feel particularly strongly, I just thought it might be an oversight


-- 
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-rs] alamb commented on issue #1578: Require CI Checks

Posted by GitBox <gi...@apache.org>.
alamb commented on issue #1578:
URL: https://github.com/apache/arrow-rs/issues/1578#issuecomment-1101732376

   > I'll close this as I don't feel particularly strongly, I just thought it might be an oversight
   
   I think it is more like "arrow has always done it this way and it seems useful to 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.

To unsubscribe, e-mail: github-unsubscribe@arrow.apache.org

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

[GitHub] [arrow-rs] tustvold closed issue #1578: Require CI Checks

Posted by GitBox <gi...@apache.org>.
tustvold closed issue #1578: Require CI Checks
URL: https://github.com/apache/arrow-rs/issues/1578


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