Skip to content

Commit 50f243a

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 705c7b0 commit 50f243a

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
@@ -48,13 +48,12 @@ def determine_modules_for_files(filenames):
4848
['pyspark-core', 'sql']
4949
>>> [x.name for x in determine_modules_for_files(["file_not_matched_by_any_subproject"])]
5050
['root']
51-
>>> [x.name for x in determine_modules_for_files( \
52-
[".github/workflows/master.yml", "appveyor.yml"])]
51+
>>> [x.name for x in determine_modules_for_files(["appveyor.yml"])]
5352
[]
5453
"""
5554
changed_modules = set()
5655
for filename in filenames:
57-
if filename in (".github/workflows/master.yml", "appveyor.yml"):
56+
if filename in ("appveyor.yml",):
5857
continue
5958
matched_at_least_one_module = False
6059
for module in modules.all_modules:

0 commit comments

Comments
 (0)