vincbeck opened a new issue, #56738: URL: https://github.com/apache/airflow/issues/56738
### Apache Airflow version 3.1.0 ### If "Other Airflow 2/3 version" selected, which one? _No response_ ### What happened? ## Summary This issue tracks all files currently failing the `mypy-providers` test. The failures are grouped by directory for easier navigation and fixing. ## How to generate the errors - Apply changes from https://github.com/apache/airflow/pull/56212 locally - Run `breeze ci-image build --python 3.10 --upgrade-to-newer-dependencies` - Run `prek --color always --verbose --hook-stage manual "mypy-providers" --all-files` reveals numerous type errors throughout the codebase ## Error Breakdown by Directory ### providers/fab/ - `providers/fab/src/airflow/providers/fab/auth_manager/models/__init__.py` (5 errors) - `providers/fab/tests/unit/fab/auth_manager/models/test_user_model.py` (1 error) ### providers/standard/ - `providers/standard/src/airflow/providers/standard/utils/sensor_helper.py` (4 errors) - `providers/standard/src/airflow/providers/standard/triggers/external_task.py` (1 error) - `providers/standard/src/airflow/providers/standard/sensors/external_task.py` (1 error) - `providers/standard/src/airflow/providers/standard/operators/latest_only.py` (5 errors) - `providers/standard/tests/unit/standard/operators/test_hitl.py` (12 errors) ### providers/neo4j/ - `providers/neo4j/src/airflow/providers/neo4j/hooks/neo4j.py` (1 error) ### providers/edge3/ - `providers/edge3/src/airflow/providers/edge3/models/edge_logs.py` (7 errors) - `providers/edge3/src/airflow/providers/edge3/models/edge_job.py` (12 errors) - `providers/edge3/src/airflow/providers/edge3/models/edge_worker.py` (20 errors) - `providers/edge3/src/airflow/providers/edge3/worker_api/routes/jobs.py` (15 errors) - `providers/edge3/src/airflow/providers/edge3/worker_api/routes/ui.py` (16 errors) - `providers/edge3/src/airflow/providers/edge3/worker_api/routes/worker.py` (15 errors) - `providers/edge3/src/airflow/providers/edge3/executors/edge_executor.py` (18 errors) ### providers/databricks/ - `providers/databricks/src/airflow/providers/databricks/hooks/databricks_base.py` (6 errors) - `providers/databricks/src/airflow/providers/databricks/plugins/databricks_workflow.py` (1 error) - `providers/databricks/src/airflow/providers/databricks/hooks/databricks_sql.py` (1 error) ### providers/opensearch/ - `providers/opensearch/src/airflow/providers/opensearch/log/os_task_handler.py` (4 errors) ### providers/cncf/kubernetes/ - `providers/cncf/kubernetes/src/airflow/providers/cncf/kubernetes/executors/kubernetes_executor.py` (3 errors) ### providers/openlineage/ - `providers/openlineage/src/airflow/providers/openlineage/utils/sql.py` (4 errors) - `providers/openlineage/src/airflow/providers/openlineage/utils/utils.py` (1 error) - `providers/openlineage/src/airflow/providers/openlineage/plugins/listener.py` (4 errors) ### providers/imap/ - `providers/imap/src/airflow/providers/imap/hooks/imap.py` (4 errors) ### providers/elasticsearch/ - `providers/elasticsearch/src/airflow/providers/elasticsearch/log/es_task_handler.py` (4 errors) ### providers/common/sql/ - `providers/common/sql/src/airflow/providers/common/sql/dialects/dialect.py` (3 errors) - `providers/common/sql/src/airflow/providers/common/sql/hooks/sql.py` (1 error) ### providers/amazon/ - `providers/amazon/tests/system/amazon/aws/example_dms.py` (2 errors) - `providers/amazon/tests/system/amazon/aws/example_dms_serverless.py` (2 errors) - `providers/amazon/src/airflow/providers/amazon/aws/hooks/redshift_sql.py` (4 errors) - `providers/amazon/src/airflow/providers/amazon/aws/hooks/athena_sql.py` (1 error) ### providers/teradata/ - `providers/teradata/src/airflow/providers/teradata/hooks/teradata.py` (4 errors) ### providers/apache/drill/ - `providers/apache/drill/src/airflow/providers/apache/drill/hooks/drill.py` (1 error) ## How to Contribute 1. **Pick a file**: Choose an unchecked file from the list above 2. **Check the box**: Update this issue to mark the file as being worked on 3. **Run MyPy locally**: Follow the steps mentioned above the generate the errors. You can run `prek --color always --verbose --hook-stage manual "mypy-providers" <file/directory>` to only run mypy against the file/directory you are working on 4. **Fix the errors**: Address the type errors in your chosen file 5. **Test your changes**: Ensure tests still pass 6. **Submit a PR**: Create a pull request with your fixes. Please do not include in your PR changes from https://github.com/apache/airflow/pull/56212 ## Guidelines - Focus on one file or directory at a time - Follow existing code patterns and conventions --- Let's work together to improve Airflow's type safety! 🚀 **Note**: This issue was generated by running MyPy on the current codebase. The error counts may change as the codebase evolves. ### What you think should happen instead? _No response_ ### How to reproduce - Apply changes from https://github.com/apache/airflow/pull/56212 locally - Run `breeze ci-image build --python 3.10 --upgrade-to-newer-dependencies` - Run `prek --color always --verbose --hook-stage manual "mypy-providers" --all-files` reveals numerous type errors throughout the codebase ### Operating System Macos ### Versions of Apache Airflow Providers _No response_ ### Deployment Other ### Deployment details _No response_ ### Anything else? _No response_ ### Are you willing to submit PR? - [ ] Yes I am willing to submit a PR! ### Code of Conduct - [x] I agree to follow this project's [Code of Conduct](https://github.com/apache/airflow/blob/main/CODE_OF_CONDUCT.md) -- 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]
