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

   ## Why
   While I was adding cross-worktree image reuse in #71886, some of the 
functions' name makes the code tracing harder:
   - `rebuild_or_pull_ci_image_if_needed`: it was introduced in #24402, when 
passing an image tag made the helper pull that CI image. The pull path was 
later removed, but the old name remained. 
   - `check_if_image_building_is_needed` calls `should_we_run_the_build`; the 
similar names hide that the inner helper checks source changes and asks the 
user for confirmation.
   
   ## What
   
   Rename the helpers to `build_ci_image_if_needed` and 
`confirm_build_if_sources_changed`. Update their call sites, docstrings, mocks, 
and test names without changing behavior.
   
   related: #71886
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code and Codex (GPT-5)
   
   Generated-by: Claude Code and Codex (GPT-5) following [the 
guidelines](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#gen-ai-assisted-contributions)


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