potiuk opened a new pull request, #76837:
URL: https://github.com/apache/airflow/pull/76837

   ## Why
   
   One of the points that cancelled the AIP-120 vote: fork CI tests the tip of 
the contributor's
   branch, while project CI tests the PR merged into the target branch, so a 
gated PR could go
   green on a stale base and merge on that result.
   
   ## What
   
   - When a fork run turns green, the fork CI monitor compares the head commit 
with the base
     branch: the merge base the run tested, how many commits the base branch 
has since, and which
     of the PR's files the base branch changed since.
   - `CI passed` is reported only when the base is fewer than 200 commits 
behind and changed none
     of the PR's files. Otherwise it is posted as pending with "Rebase on 
<base> and push", and the
     state comment lists the overlapping files and the rebase commands.
   - Every green state comment and the `CI passed` description name the base 
commit the run was
     tested on.
   - Testing the merge commit in the fork itself was considered and rejected 
(every checkout of the
     fork run would change, there is no merge ref before the PR exists, GitHub 
recomputes it
     asynchronously, and the monitor would have to trust the run's own report); 
the reasons are in
     the design doc's "Known limitations".
   - The check runs once per head commit, like project CI's `pull_request` run, 
and no paths are
     exempt from the overlap check; both can be revisited if rebase churn 
becomes a problem.
   
   related: https://lists.apache.org/thread/5603zwgpj7pgj6j979hbgpr72xsjbbws
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 5)
   
   Generated-by: Claude Code (Opus 5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)
   
   🤖 Generated with [Claude Code](https://claude.com/claude-code)
   


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

  • [PR] Report fork CI's tested ... potiuk (via GitHub) via commits

Reply via email to