Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Commit fbc5ff6

Browse files
committed
patchcheck: don’t talk about the test suite when no code file were changed.
The line about the test suite will still get printed for changes in Tools for example, which aren’t covered by the test suite, but it’s not a big deal IMO.
1 parent 28d39a0 commit fbc5ff6

1 file changed

Lines changed: 3 additions & 2 deletions

File tree

Tools/scripts/patchcheck.py

Lines changed: 3 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -157,8 +157,9 @@ def main():
157157
reported_news(special_files)
158158

159159
# Test suite run and passed.
160-
print()
161-
print("Did you run the test suite?")
160+
if python_files or c_files:
161+
print()
162+
print("Did you run the test suite?")
162163

163164

164165
if __name__ == '__main__':

0 commit comments

Comments
 (0)