sfirke commented on PR #35896:
URL: https://github.com/apache/superset/pull/35896#issuecomment-3468667523

   Looks like you are adding Pillow to the base Docker image. I'm fine with 
that, and once this PR is in a major release I'll update 
https://superset.apache.org/docs/installation/docker-builds/ to remove Pillow 
as an add-on.
   
   But are Playwright and Chrome still not installed in the `python-common` or 
`lean` layers by default? If that's still the case, then does it help to add 
Pillow here? I think of Pillow as being needed for Alerts & Reports but if 
someone has to install Playwright and Chrome anyway would they install it then? 
Just wondering, and maybe I'm off-base here.


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


---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]

Reply via email to