potiuk commented on PR #65199: URL: https://github.com/apache/airflow/pull/65199#issuecomment-4299712333
@harshadkhetpal Thank you for taking the time to open a PR — closing this one without approving CI because the changes don't match the PR title and don't look intended for the upstream `apache/airflow` repository: - The title is *"fix: replace bare except with Exception"*, but the actual diff is a personal fork-intro section added to `README.md` (under `## 🧑💻 My Fork — Harshad Khetpal (DevOps / MLOps Engineer)`) plus a standalone `dags/ml_pipeline_dag.py`. Those changes make sense in your own fork — and you are welcome to keep them there — but they aren't changes that the upstream `apache/airflow` repository can merge. If you'd like to contribute upstream, we'd warmly welcome it. A good starting point is the [contributors guide](https://github.com/apache/airflow/blob/main/contributing-docs/README.rst) and the [first-time-contributor issues](https://github.com/apache/airflow/contribute). Pick something concrete (a typo fix, a docstring clarification, a small bug fix from a GitHub issue), make sure the PR title and diff describe the same thing, and we'll review it. Closing this one — please reopen with a PR that matches its title, or open a fresh one against a specific issue. Thanks! -- 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]
