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

   
   * related: #56150
   
   ## Why
   
   Previously, the default value parser logic in `AirflowConfigParser` did not 
check for `version_deprecated ` or `deprecation_reason ` for default 
`config.yaml`
   
   This means that if we really deprecated a `[config/key]`, like the case of 
https://github.com/apache/airflow/pull/56150#discussion_r2734687449 (not 
renaming or moving to other section like AF2 -to AF3), the `conf.get` will 
still get default value from the `config.yaml`.
   
   From my perspective, the default value parser should respect deprecated 
options (the option that provide either `version_deprecated` or 
`deprecation_reason`) and ignore it.
   
   ## What
   
   - Update the config parser to **respect deprecated options for default value 
parser**
   - Extract the `configure_parser_from_configuration_description` logic to 
shared lib
   
   ##### Was generative AI tooling used to co-author this PR?
   
   - [x] Yes (please specify the tool below)
   
   Generated-by: Copilot (ChatGPT 4.1), Gemini-CLI (auto model) 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