potiuk commented on a change in pull request #20200:
URL: https://github.com/apache/airflow/pull/20200#discussion_r771994173
##########
File path: .github/workflows/build-images.yml
##########
@@ -224,7 +224,7 @@ jobs:
rm -rf "scripts/ci"
mv "main-airflow/scripts/ci" "scripts"
- name: "Free space"
- run: ./scripts/ci/tools/free_space.sh
+ run: ./scripts/ci/tools/freespace.py
Review comment:
Comment here: for all the "freespace.py" script, you will also have to
install dependenciesl (those that are specified in setup.py and setup.cfg in
dev/breeze
The setup-python action supports caching dependencies:
* https://github.com/actions/setup-python#caching-packages-dependencies
So you'd have to add something like I've done here:
https://github.com/apache/airflow/blob/4ac35d723b73d02875d56bf000aafd2235ef0f4a/.github/workflows/ci.yml#L240
for every "setup-python" step in the ci.yml.
--
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]
