Skip to content

Commit f00a73f

Browse files
committed
[SPARK-32496][INFRA] Include GitHub Action file as the changes in testing
### What changes were proposed in this pull request? apache#26556 excluded `.github/workflows/master.yml`. So tests are skipped if the GitHub Actions configuration file is changed. As of SPARK-32245, we now run the regular tests via the testing script. We should include it to test to make sure GitHub Actions build does not break due to some changes such as Python versions. ### Why are the changes needed? For better test coverage in GitHub Actions build. ### Does this PR introduce _any_ user-facing change? No, dev-only. ### How was this patch tested? GitHub Actions in this PR will test. Closes apache#29305 from HyukjinKwon/SPARK-32496. Authored-by: HyukjinKwon <[email protected]> Signed-off-by: HyukjinKwon <[email protected]>
1 parent 64780e6 commit f00a73f

File tree

1 file changed

+2
-3
lines changed

1 file changed

+2
-3
lines changed

dev/run-tests.py

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -49,13 +49,12 @@ def determine_modules_for_files(filenames):
4949
['pyspark-core', 'sql']
5050
>>> [x.name for x in determine_modules_for_files(["file_not_matched_by_any_subproject"])]
5151
['root']
52-
>>> [x.name for x in determine_modules_for_files( \
53-
[".github/workflows/master.yml", "appveyor.yml"])]
52+
>>> [x.name for x in determine_modules_for_files(["appveyor.yml"])]
5453
[]
5554
"""
5655
changed_modules = set()
5756
for filename in filenames:
58-
if filename in (".github/workflows/master.yml", "appveyor.yml"):
57+
if filename in ("appveyor.yml",):
5958
continue
6059
matched_at_least_one_module = False
6160
for module in modules.all_modules:

0 commit comments

Comments
 (0)