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 2021/06/22 10:19:40 UTC

[GitHub] [airflow] ashb commented on a change in pull request #16577: Exclude ``yarn.lock`` from release packages

ashb commented on a change in pull request #16577:
URL: https://github.com/apache/airflow/pull/16577#discussion_r656081985



##########
File path: airflow/www/compile_assets.sh
##########
@@ -31,4 +31,9 @@ fi
 yarn install --frozen-lockfile
 yarn run build
 
-find package.json yarn.lock static/css static/js -type f | sort | xargs md5sum > "${MD5SUM_FILE}"
+YARN_LOCK_FILE="yarn.lock"
+if [[ -f "$YARN_LOCK_FILE" ]]; then

Review comment:
       We shouldn't be running this script if there is no lock file -- else we have run `yarn install --frozen-lockfile` which will (I hope!) fail.




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

For queries about this service, please contact Infrastructure at:
users@infra.apache.org