shahar1 opened a new pull request, #66978:
URL: https://github.com/apache/airflow/pull/66978
When a `logging` call has more positional arguments than the format string
has `%`-placeholders, the extra arguments are silently dropped and the intended
information never appears in the log. Enable `PLE1205`
(`logging-too-many-args`) to make this a lint error, and fix all 12 existing
instances.
## Changes
- **\`pyproject.toml\`**: add `PLE1205` to `extend-select`.
- **\`airflow-core\`** (1): add missing `%s` placeholder for the caught
`OSError` in `file_task_handler.py`.
- **\`provider:amazon\`** (4):
- `redshift_data.py`: collapse two-string call into `"Invalid
poll_interval: %s", poll_interval`
- `ssm.py`: merge accidentally split message strings into one format
string with `%s`
- `task_log_fetcher.py` (×2): add `%s` for the captured exception
- **\`provider:google\`** (7):
- `compute.py` (×2): add `%s` so `self.body` is actually logged
- `drive.py`: add `%s` for the `GoogleApiClientError`
- Four system-test example files using `{}` placeholders (logging uses
`%s`, not `.format()` syntax)
---
##### Was generative AI tooling used to co-author this PR?
- [X] Yes — Claude Code (Sonnet 4.6)
Generated-by: Claude Code (Sonnet 4.6) 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]
