You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@airflow.apache.org by "Bowrna (via GitHub)" <gi...@apache.org> on 2025/05/29 10:31:27 UTC

Re: [PR] Verify decorators parameter in __init__.pyi matches operator [airflow]

Bowrna commented on PR #51048:
URL: https://github.com/apache/airflow/pull/51048#issuecomment-2918986713

   @uranusjr There are some complexities in moving to pre-commit. As unit tests, they run within the virtualenv and are context-aware, running without issue. But to run as part of a pre-commit hook, setting up the path has to be done. even when done, 
   
   ```module = __import__(module_path, fromlist=[class_name])```
   This part of the code in the test loads and runs the module in real-time, triggering a missing dependency issue at runtime. how do you think this could be better handled?


-- 
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: commits-unsubscribe@airflow.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org