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:

Reply via email to