potiuk commented on issue #74908:
URL: https://github.com/apache/airflow/issues/74908#issuecomment-6452390161

   The same gap now applies to a second comment. #75814 adds a "Fork CI stopped 
running" comment, posted when a pull request whose fork CI had worked (for 
example `awaiting_push`, `failed` or `green`) moves to `setup_required` because 
the fork's `Tests (AMD)` workflow was disabled or removed. Like the nudge, it 
is posted after the state comment records `setup_required`. If posting it 
fails, the next run sees `setup_required` and does not post it again. The 
author then gets no notice before the inactivity closure.
   
   Whichever fix lands here (posting before recording the new state, or a 
pending-notification flag in the marker) should cover both comments. Both sites 
in `classify` in `dev/breeze/src/airflow_breeze/utils/fork_ci_reconcile.py` 
link to this issue.
   
   ---
   Drafted-by: Claude Code (Claude Opus 5); reviewed by @potiuk before posting
   


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

Reply via email to