Skip to content

Commit 3e26492

Browse files
committed
[SPARK-29199][INFRA] Add linters and license/dependency checkers to GitHub Action
### What changes were proposed in this pull request? This PR aims to add linters and license/dependency checkers to GitHub Action. This excludes `lint-r` intentionally because https://github.com/actions/setup-r is not ready. We can add that later when it becomes available. ### Why are the changes needed? This will help the PR reviews. ### Does this PR introduce any user-facing change? No. ### How was this patch tested? See the GitHub Action result on this PR. Closes #25879 from dongjoon-hyun/SPARK-29199. Authored-by: Dongjoon Hyun <dhyun@apple.com> Signed-off-by: Dongjoon Hyun <dhyun@apple.com>
1 parent 81b6f11 commit 3e26492

File tree

1 file changed

+26
-0
lines changed

1 file changed

+26
-0
lines changed

.github/workflows/master.yml

Lines changed: 26 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -32,3 +32,29 @@ jobs:
3232
export MAVEN_OPTS="-Xmx2g -XX:ReservedCodeCacheSize=1g -Dorg.slf4j.simpleLogger.defaultLogLevel=WARN"
3333
export MAVEN_CLI_OPTS="--no-transfer-progress"
3434
./build/mvn $MAVEN_CLI_OPTS -DskipTests -Pyarn -Pmesos -Pkubernetes -Phive -Phive-thriftserver -P${{ matrix.hadoop }} -Phadoop-cloud -Djava.version=${{ matrix.java }} package
35+
36+
37+
lint:
38+
runs-on: ubuntu-latest
39+
name: Linters
40+
steps:
41+
- uses: actions/checkout@master
42+
- uses: actions/setup-java@v1
43+
with:
44+
java-version: '11'
45+
- uses: actions/setup-python@v1
46+
with:
47+
python-version: '3.x'
48+
architecture: 'x64'
49+
- name: Scala
50+
run: ./dev/lint-scala
51+
- name: Java
52+
run: ./dev/lint-java
53+
- name: Python
54+
run: |
55+
pip install flake8 sphinx numpy
56+
./dev/lint-python
57+
- name: License
58+
run: ./dev/check-license
59+
- name: Dependencies
60+
run: ./dev/test-dependencies.sh

0 commit comments

Comments
 (0)