meel-ayush opened a new pull request, #76854:
URL: https://github.com/apache/airflow/pull/76854

   ### Description
   
   Part of **AIP-120**:
   
   In `breeze ci audit`, when `--fix` runs to remediate a missing or 
misconfigured `origin` remote, it hardcoded HTTPS URLs 
(`https://github.com/{login}/airflow.git`).
   
   If a contributor cloned using SSH (`[email protected]:...` or an alias from 
`~/.ssh/config`), running `--fix` rewrote their `origin` remote to HTTPS. This 
broke subsequent `git push` commands with SSH keys and prompted for credentials.
   
   This change:
   * Inspects existing remote URLs using `GITHUB_REMOTE_URL` to detect the 
remote protocol.
   * Preserves the SSH prefix (including custom host aliases and ports) when 
generating `origin` commands.
   * Keeps standard HTTPS URLs for HTTPS clones.
   
   ### Tests
   
   - Unit tests added in `dev/breeze/tests/test_ci_audit.py` covering SSH 
origin repointing, SSH origin addition, custom SSH host aliases, and HTTPS 
backward compatibility.
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [ ] Yes (please specify the tool below)


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