potiuk commented on PR #977:
URL: https://github.com/apache/airflow-site/pull/977#issuecomment-1986919423
This looks cool, I already made my comments to the content of the videos and
voice-over (being a professional lector I pay attention to some quality
problems ;) - and it's corrected now (thanks @kentdanas and @TJaniF !)
It's cool and nicely organised. Also, I noticed a bit too much "Astronomer"
in the original videos - there were tabs visible and prominent Astronomer
references. What's left is pretty ok for me, we have a mixture of "pure OSS"
videos and few with slightly visible Astronomer reference, but the level it is
now, is pretty ok for me.
I really love the discoverability of cases it gives and the way that we are
trying to reach out using various media.
One thing I am **slightly** worried about is the weight it adds to our
website to embed those videos in our git. This adds significant weight to it
and we know we already have huge problems in `airflow-site` size and it might
break our CI process even more eventually (at least until we possibly split out
airflow-site) cc: @utkarsharma2.
Maybe it's a good idea to upload the videos to youtube and host them from
there ?
--
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-unsubscr...@airflow.apache.org
For queries about this service, please contact Infrastructure at:
us...@infra.apache.org
