You are viewing a plain text version of this content. The canonical link for it is here.

Posted to commits@airflow.apache.org by GitBox <gi...@apache.org> on 2022/10/28 04:23:39 UTC

[GitHub] [airflow] o-nikolas commented on issue #27338: scripts/tools/initialize_virtualenv.py calling .exists() on str

o-nikolas commented on issue #27338:
URL: https://github.com/apache/airflow/issues/27338#issuecomment-1294435687

   Thanks for the bug report @rkarish!
   
   I've assigned you the task since you checked that you're willing to submit a PR :smiley: 
   
   >  I believe this should be os.path.exists(airflow_home) instead
   
   Looking at the function stub for `clean_up_airflow_home` it looks like that code is expecting a `Path` object, but it's getting a string. So I think the better fix is to update the calling code to pass in a Path object instead of a string as it is now. 
   


-- 
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