Re: [PR] Auto-triage: flag area:UI PRs missing screenshots/demo from new contributors [airflow]

["potiuk (via GitHub)" <gi...@apache.org>]

potiuk commented on code in PR #64516:
URL: https://github.com/apache/airflow/pull/64516#discussion_r3021435292


##########
dev/breeze/src/airflow_breeze/commands/pr_commands.py:
##########
@@ -2152,6 +2152,9 @@ def _compute_default_action(
     elif not has_ci_failures and not has_conflicts and has_unresolved_comments:
         # CI passes, no conflicts, no LLM issues — only unresolved review comments; just add a comment
         action = TriageAction.COMMENT
+    elif not has_ci_failures and not has_conflicts and not has_unresolved_comments:
+        # Only soft violations (e.g. missing UI demo) — just comment, don't convert to draft
+        action = TriageAction.COMMENT

Review Comment:
   Yeah. This  one is not necessary - also "Draft" is a suggestion - can be changed to comment by maintainer..



-- 
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: commits-unsubscribe@airflow.apache.org

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