feluelle commented on a change in pull request #14998:
URL: https://github.com/apache/airflow/pull/14998#discussion_r601176365
##########
File path: scripts/ci/libraries/_initialization.sh
##########
@@ -186,6 +186,19 @@ function
initialization::initialize_available_integrations() {
FILES_FOR_REBUILD_CHECK=()
# Determine which files trigger rebuild check
+#
+# !!!! IMPORTANT NOTE !!!!!!!!!!!!!!!!!!!!!!!!1
+# When you add files here, please make sure to not add files
+# with the same name. And if you do - make sure that files with the
+# same name are stored in directories with different name. For
+# example we hava two package.json files here, but they are in
+# directories with different names (`wwww` and `ui`).
+# The problem is that md5 hashes of those files are stored in
+# ./build/directory in the same directory as <PARENT_DIR>-<FILE>.md5sum .
+# For example md5sum of the `airflow/www/package.json` file is stored
+# as `www-package.json` and `airflow/ui/package.json` as `ui-package.json`,
+# The file list here changes extremely rarely.
+# !!!! IMPORTANT NOTE !!!!!!!!!!!!!!!!!!!!!!!!1
Review comment:
```suggestion
# !!!!!!!!!! IMPORTANT NOTE !!!!!!!!!!
# When you add files here, please make sure to not add files
# with the same name. And if you do - make sure that files with the
# same name are stored in directories with different name. For
# example we have two package.json files here, but they are in
# directories with different names (`www` and `ui`).
# The problem is that md5 hashes of those files are stored in
# `./build/directory` in the same directory as <PARENT_DIR>-<FILE>.md5sum.
# For example md5sum of the `airflow/www/package.json` file is stored
# as `www-package.json` and `airflow/ui/package.json` as `ui-package.json`,
# The file list here changes extremely rarely.
# !!!!!!!!!! IMPORTANT NOTE !!!!!!!!!!
```
--
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:
[email protected]
