Skip to content

Commit be0f610

Browse files
committed
Revert "fix(ci): skip doc-only files in module change detection"
This reverts commit 94a0df2.
1 parent 95c0785 commit be0f610

1 file changed

Lines changed: 2 additions & 21 deletions

File tree

scripts/detect_modules.py

Lines changed: 2 additions & 21 deletions
Original file line numberDiff line numberDiff line change
@@ -107,31 +107,12 @@ def get_impacted_modules(
107107
return impacted
108108

109109

110-
def is_code_relevant(file_path: str) -> bool:
111-
"""Check if a changed file is relevant for testing (not just docs/static)."""
112-
p = Path(file_path)
113-
parts = p.parts
114-
# Skip top-level README
115-
if p.name == "README.rst":
116-
return False
117-
# Skip readme fragments directory
118-
if len(parts) > 1 and parts[1] == "readme":
119-
return False
120-
# Skip static/description (auto-generated index.html, icons, etc.)
121-
if len(parts) > 2 and parts[1] == "static" and parts[2] == "description":
122-
return False
123-
return True
124-
125-
126110
def extract_modules_from_files(changed_files: list[str], all_modules: set[str]) -> set[str]:
127-
"""Extract module names from changed file paths.
128-
129-
Only considers code-relevant files (ignores READMEs, static descriptions).
130-
"""
111+
"""Extract module names from changed file paths."""
131112
modules: set[str] = set()
132113
for file_path in changed_files:
133114
parts = Path(file_path).parts
134-
if parts and parts[0] in all_modules and is_code_relevant(file_path):
115+
if parts and parts[0] in all_modules:
135116
modules.add(parts[0])
136117
return modules
137118

0 commit comments

Comments
 (0)