potiuk commented on code in PR #56187:
URL: https://github.com/apache/airflow/pull/56187#discussion_r2523839319


##########
task-sdk/pyproject.toml:
##########
@@ -77,6 +77,14 @@ dependencies = [
     # Start of shared logging dependencies
     "pygtrie>=2.5.0",
     # End of shared logging dependencies
+    # Start of shared observability dependencies

Review Comment:
   Actually @xBis7 -> one more question here. It seems that observability has 
`sqlalchemy` dependency that is added here by prek -> and I do not see any 
reason why sqlalchemy should be a dependency of observability? Is it an 
accidental addition? 



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

Reply via email to