From e2628aa60ca6f0765547dbf7e935f7c96a0b8c41 Mon Sep 17 00:00:00 2001
From: Sirz Benjie <142067137+SirzBenjie@users.noreply.github.com>
Date: Sat, 15 Mar 2025 23:06:20 -0400
Subject: [PATCH] Apply suggestions from kev's review

Co-authored-by: NightKev <34855794+DayKev@users.noreply.github.com>
---
 .github/workflows/tests.yml | 12 ++++++------
 1 file changed, 6 insertions(+), 6 deletions(-)

diff --git a/.github/workflows/tests.yml b/.github/workflows/tests.yml
index 0dbbcddf4eb..d6030383d5c 100644
--- a/.github/workflows/tests.yml
+++ b/.github/workflows/tests.yml
@@ -12,7 +12,7 @@ on:
       # src and test files
       - "src/**"
       - "test/**"
-      - "public/**" # PLEASE REVIEW -- Does it make sense to include this? Can modifying these break tests? (REMOVE COMMENT AFTER REVIEW)
+      - "public/**"
       # Workflows that can impact tests
       - ".github/workflows/test*.yml"
       # top-level files
@@ -21,7 +21,7 @@ on:
       - "vite.*.ts" # vite.config.ts, vite.vitest.config.ts, vitest.workspace.ts
       - "tsconfig*.json" # tsconfig.json tweaking can impact compilation
       - "global.d.ts" # I think this can impact tests? (REMOVE COMMENT AFTER REVIEW)
-      - ".env.*" # PLEASE REVIEW -- can these impact test behavior?
+      - ".env.*"
       # Blanket negations for files that cannot impact tests
       - "!**/*.py" # No .py files
       - "!**/*.sh" # No .sh files
@@ -38,7 +38,7 @@ on:
       # src and test files
       - "src/**"
       - "test/**"
-      - "public/**" # PLEASE REVIEW -- Does it make sense to include this? Can modifying these break tests? (REMOVE COMMENT AFTER REVIEW)
+      - "public/**"
       # Workflows that can impact tests
       - ".github/workflows/test*.yml"
       # top-level files
@@ -47,13 +47,13 @@ on:
       - "vite*" # vite.config.ts, vite.vitest.config.ts, vitest.workspace.ts
       - "tsconfig*.json" # tsconfig.json tweaking can impact compilation
       - "global.d.ts" # I think this can impact tests? (REMOVE COMMENT AFTER REVIEW)
-      - ".env.*" # PLEASE REVIEW -- can these impact test behavior?
+      - ".env.*"
       # Blanket negations for files that cannot impact tests
       - "!**/*.py" # No .py files
       - "!**/*.sh" # No .sh files
       - "!**/*.md" # No .md files
-      - "!**/*.png" # TODO: Can the deletion of these files break tests? (REMOVE COMMENT AFTER REVIEW)
-      - "!**/*.jpg" # TODO: Can the deletion of these files break tests? (REMOVE COMMENT AFTER REVIEW)
+      - "!**/*.png"
+      - "!**/*.jpg"
       - "!**/.git*" # .gitkeep and family
   merge_group:
     types: [checks_requested]