SUMIN/HuggingFace_transformer
1
0
Fork 0
Code Issues Pull Requests Actions 7 Packages Projects Releases Wiki Activity

Revert "Only add files with modification outside doc blocks" (#23371)

Browse Source 
fix

Co-authored-by: ydshieh <ydshieh@users.noreply.github.com>
This commit is contained in:
Yih-Dar
2023-05-15 14:28:36 +02:00
committed by GitHub
parent c94f7a1cce
commit 569a97adb2
1 changed files with 3 additions and 16 deletions
Download Patch File Download Diff File Expand all files Collapse all files

19
utils/tests_fetcher.py
View File

@@ -225,23 +225,10 @@ def get_diff_for_py_and_mdx_files(repo, base_commit, commits):
for commit in commits: for commit in commits:
for diff_obj in commit.diff(base_commit): for diff_obj in commit.diff(base_commit):
# We always add new python files # We always add new python files
if diff_obj.change_type in ["A"] and (diff_obj.b_path.endswith(".py") or diff_obj.b_path.endswith(".mdx")): if diff_obj.change_type in ["A", "M", "R"] and (
code_diff.append(diff_obj.b_path) diff_obj.b_path.endswith(".py") or diff_obj.b_path.endswith(".mdx")
# Now for modified files
elif (
diff_obj.change_type in ["M", "R"]
and diff_obj.b_path.endswith(".py")
or diff_obj.b_path.endswith(".mdx")
): ):
# In case of renames, we'll look at the tests using both the old and new name. code_diff.append(diff_obj.b_path)
if diff_obj.a_path != diff_obj.b_path:
code_diff.extend([diff_obj.a_path, diff_obj.b_path])
else:
# Otherwise, we check modifications are in code and not docstrings.
if diff_is_docstring_only(repo, commit, diff_obj.b_path):
print(f"Ignoring diff in {diff_obj.b_path} as it only concerns docstrings or comments.")
else:
code_diff.append(diff_obj.a_path)
return code_diff return code_diff