comphead commented on code in PR #26683: URL: https://github.com/apache/datafusion/pull/26683#discussion_r4470034671
########## .github/workflows/fork_ci_status.yml: ########## @@ -7,18 +7,20 @@ # A pull request from a fork to `main` runs `rust.yml` on the fork. This # workflow mirrors that run as one `Rust (fork)` check on the pull request, so # reviewers see the result without opening the fork. It never checks out or -# runs code from the pull request: on `pull_request_target` it reads the fork's -# public run list and creates the check, and on a schedule it updates the -# active or missing runs of recently updated pull requests plus a rotating -# slice of all open pull requests. The check is informational. The merge +# runs code from the pull request: on `workflow_run` of +# `pull_request_signal.yml` it reads the fork's public run list and creates the +# check (`pull_request_target` is disabled by ASF infra policy), and on a +# schedule it updates the active or missing runs of recently updated pull +# requests plus a rotating slice of all open pull requests. The check is informational. The merge Review Comment: Fixed in 4c3c2e980. Re-wrapped, and I dropped the parenthetical that the header of the signal workflow already explains. -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected] --------------------------------------------------------------------- To unsubscribe, e-mail: [email protected] For additional commands, e-mail: [email protected]
