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

   ## Summary
   
   - Cherry-picks #66178 (`main` commit `9b9789ae6b`,
     "Use contextlib.suppress instead of try-except-pass in providers") to
     v3-2-test, dropping the two files that don't exist in v3-2-test
     (`providers/common/ai/.../durable/storage.py` and
     `providers/ssh/.../tunnel.py`) and the test example file
     (`example_s3_tables.py`) that also isn't on v3-2-test.
   - Adds a follow-up commit applying the same SIM105 fix to
     `providers/apache/spark/.../hooks/spark_jdbc_script.py`. On main this
     file was rewritten in #64174 to raise an explicit
     `AirflowOptionalProviderFeatureException`, but that change is out of
     scope for v3-2-test, so we use the minimum SIM105-compliant edit
     (`contextlib.suppress(ImportError)`).
   
   The SIM105 rule was re-enabled on v3-2-test by #66264 (the backport of
   main's #66193). #66264 cherry-picked the rule re-enablement but did
   **not** include the provider-side fixes that #66178 had already done on
   main, so v3-2-test's static-checks job has been failing with 8 SIM105
   hits across these 7 files since #66264 merged. This PR closes that gap.
   
   ## Test plan
   
   - [ ] `prek run ruff --all-files` passes locally (verified — 8 SIM105
         hits resolved)
   - [ ] CI `Static checks` job goes green on v3-2-test
   
   ---
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [X] Yes — Claude Code (Opus 4.7)
   
   Generated-by: Claude Code (Opus 4.7) 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