Skip to content

Commit

Permalink
Update Breeze UI Files to match pnpm lock file (#43699)
Browse files Browse the repository at this point in the history
For the CI, we just copied over the www files over to the UI_FILES, but as caught [here](#43633 (comment)), pnpm's lock file is yaml and wasn't being caught.


Now, we will just have all files inside of `airflow/ui` count. Then, changes to the html file,`.prettierrc`, or anything else also don't force the CI to run everything.

Also, added the pnpm lock yaml file to the javascript production files check.
  • Loading branch information
bbovenzi authored Nov 5, 2024
1 parent dd9b735 commit 71b8d9f
Showing 1 changed file with 2 additions and 6 deletions.
8 changes: 2 additions & 6 deletions dev/breeze/src/airflow_breeze/utils/selective_checks.py
Original file line number Diff line number Diff line change
Expand Up @@ -154,6 +154,7 @@ def __hash__(self):
FileGroupForCi.JAVASCRIPT_PRODUCTION_FILES: [
r"^airflow/.*\.[jt]sx?",
r"^airflow/.*\.lock",
r"^airflow/ui/.*\.yaml$",
],
FileGroupForCi.API_TEST_FILES: [
r"^airflow/api/",
Expand Down Expand Up @@ -189,12 +190,7 @@ def __hash__(self):
r"^chart/values\.schema\.json",
r"^chart/values\.json",
],
FileGroupForCi.UI_FILES: [
r"^airflow/ui/.*\.ts[x]?$",
r"^airflow/ui/.*\.js[x]?$",
r"^airflow/ui/[^/]+\.json$",
r"^airflow/ui/.*\.lock$",
],
FileGroupForCi.UI_FILES: [r"^airflow/ui/"],
FileGroupForCi.LEGACY_WWW_FILES: [
r"^airflow/www/.*\.ts[x]?$",
r"^airflow/www/.*\.js[x]?$",
Expand Down

0 comments on commit 71b8d9f

Please sign in to comment.