This is an automated email from the ASF dual-hosted git repository.
kaxilnaik pushed a commit to branch main
in repository https://gitbox.apache.org/repos/asf/airflow.git
The following commit(s) were added to refs/heads/main by this push:
new 5ae3eaf85ca chore(module-loading): remove outdated TODO and clarify
import_string docstring (#56713)
5ae3eaf85ca is described below
commit 5ae3eaf85ca65afdf53468e336eefe93d9d8d52c
Author: Ranuga <[email protected]>
AuthorDate: Thu Oct 16 20:31:21 2025 +0530
chore(module-loading): remove outdated TODO and clarify import_string
docstring (#56713)
* feat(module-loading): add nested attribute support to import_string
* chore: update docstrings
* chore(module-loading): remove outdated TODO and clarify import_string
docstring
* chore: remove unrequired file
* chore(module-loading): remove unrequired logic
---------
Co-authored-by: Programmer-RD-AI <[email protected]>
---
task-sdk/src/airflow/sdk/module_loading.py | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/task-sdk/src/airflow/sdk/module_loading.py
b/task-sdk/src/airflow/sdk/module_loading.py
index 6b9d572a7ba..05756a79108 100644
--- a/task-sdk/src/airflow/sdk/module_loading.py
+++ b/task-sdk/src/airflow/sdk/module_loading.py
@@ -25,9 +25,10 @@ def import_string(dotted_path: str):
"""
Import a dotted module path and return the attribute/class designated by
the last name in the path.
+ Note: Only supports top-level attributes or classes.
+
Raise ImportError if the import failed.
"""
- # TODO: Add support for nested classes. Currently, it only works for
top-level classes.
try:
module_path, class_name = dotted_path.rsplit(".", 1)
except ValueError: