Skip to content

Commit b0d7dac

Browse files
committed
[KYUUBI #1726] Extract dep workflow to a single yml and disable it in other jobs
<!-- Thanks for sending a pull request! Here are some tips for you: 1. If this is your first time, please read our contributor guidelines: https://kyuubi.readthedocs.io/en/latest/community/contributions.html 2. If the PR is related to an issue in https://github.com/apache/incubator-kyuubi/issues, add '[KYUUBI #XXXX]' in your PR title, e.g., '[KYUUBI #XXXX] Your PR title ...'. 3. If the PR is unfinished, add '[WIP]' in your PR title, e.g., '[WIP][KYUUBI #XXXX] Your PR title ...'. --> ### _Why are the changes needed?_ <!-- Please clarify why the changes are needed. For instance, 1. If you add a feature, you can talk about the use case of it. 2. If you fix a bug, you can clarify why it is a bug. --> ### _How was this patch tested?_ - [ ] Add some test cases that check the changes thoroughly including negative and positive cases if possible - [ ] Add screenshots for manual tests if appropriate - [ ] [Run test](https://kyuubi.readthedocs.io/en/latest/develop_tools/testing.html#running-tests) locally before make a pull request Closes #1726 from yaooqinn/dep2. Closes #1726 c214c06 [Kent Yao] Extract dep workflow to a single yml and disable it in other jobs Authored-by: Kent Yao <yao@apache.org> Signed-off-by: Kent Yao <yao@apache.org>
1 parent 89689c0 commit b0d7dac

File tree

2 files changed

+5
-5
lines changed

2 files changed

+5
-5
lines changed

.github/workflows/master.yml

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -83,9 +83,6 @@ jobs:
8383
uses: codecov/codecov-action@v2
8484
with:
8585
verbose: true
86-
- name: Detected Dependency List Change
87-
if: ${{ matrix.java == 8 }}
88-
run: build/dependency.sh
8986
- name: Upload test logs
9087
if: failure()
9188
uses: actions/upload-artifact@v2

.github/workflows/style.yml

Lines changed: 5 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -27,8 +27,8 @@ on:
2727
- branch-*
2828

2929
jobs:
30-
lint-scala:
31-
name: Scala style check
30+
linter:
31+
name: Style and Dep check
3232
runs-on: ubuntu-20.04
3333
steps:
3434
- uses: actions/checkout@v2
@@ -37,8 +37,11 @@ jobs:
3737
with:
3838
distribution: zulu
3939
java-version: 8
40+
- name: Server dependency check
41+
run: build/dependency.sh
4042
- name: Scalastyle with Maven
4143
run: build/mvn scalastyle:check -Ptpcds -Pspark-block-cleaner -Pkubernetes -Pspark-3.1 -Pspark-3.2
4244
- name: Upload scalastyle report
4345
if: failure()
4446
run: for log in `find * -name "scalastyle-output.xml"`; do echo "=========$log========="; grep "error" $log; done
47+

0 commit comments

Comments
 (0)