potiuk commented on PR #61550: URL: https://github.com/apache/airflow/pull/61550#issuecomment-4041474884
@itayweb This PR has a few issues that need to be addressed before it can be reviewed — please see our [Pull Request quality criteria](https://github.com/apache/airflow/blob/main/contributing-docs/05_pull_requests.rst#pull-request-quality-criteria). **Issues found:** - :x: **Coherent changes / AI-Generated Code Signals**: The PR includes 6 brand-new empty `__init__.py` files under the FAB provider tree (`providers/fab/src/airflow/providers/fab/auth_manager/api_endpoints/`, `providers/fab/src/airflow/providers/fab/auth_manager/openapi/`, `providers/fab/src/airflow/providers/fab/auth_manager/schemas/`, `providers/fab/src/airflow/providers/fab/www/api_connexion/`, and two under `providers/fab/tests/`) that have absolutely nothing to do with adding a `bundle_version` parameter to the DAG run trigger endpoint. These files contain only the Apache license header and no code. This is a textbook AI-tool side-effect of generating scaffolding for unrelated paths and must be removed before review. **What to do next:** - The comment informs you what you need to do. - Fix each issue, then mark the PR as "Ready for review" in the GitHub UI - but only after making sure that all the issues are fixed. - Maintainers will then proceed with a normal review. Please address the issues above and push again. If you have questions, feel free to ask on the [Airflow Slack](https://s.apache.org/airflow-slack). -- 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]
