chen0427ok commented on PR #59291: URL: https://github.com/apache/airflow/pull/59291#issuecomment-3641072774
> While the idea is sound - we are in the middle of making task-sdk independent from airflow-core. We should not add anything in task-sdk that uses anything from `airflow-core` - so please move the code entirely to task-sdk. Thanks for the feedback! I've refactored the code to move param_config_loader entirely into the task-sdk as requested. Summary of changes: Moved files: Relocated source and test files from airflow-core to their respective paths in task-sdk. Updated imports: Changed references in param.py to use airflow.sdk.definitions.param_config_loader. Cleanup: Removed the try-except ImportError block since the dependency is now internal to the SDK. -- 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]
