-
Notifications
You must be signed in to change notification settings - Fork 321
Add GitHub Action workflow (x64) and Travis (arm64) build #524
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Changes from all commits
eb561bd
75e6841
715e6b6
981f432
41050cc
1855ce0
59c83b7
92da2b3
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,60 @@ | ||
| name: CI | ||
|
|
||
| on: | ||
| pull_request: | ||
| paths: | ||
| - '**.scala' | ||
| - '**.java' | ||
| - '**.sbt' | ||
| - '.github/workflows/**.yml' | ||
| push: | ||
| branches: | ||
| - master | ||
| paths: | ||
| - '**.scala' | ||
| - '**.java' | ||
| - '**.sbt' | ||
| - '.github/workflows/**.yml' | ||
|
|
||
| jobs: | ||
| code_format: | ||
| name: Code Format | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - name: jcheckstyle | ||
| run: ./sbt jcheckStyle | ||
| test_jdk11: | ||
| name: Test JDK11 | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: olafurpg/setup-scala@v7 | ||
| with: | ||
| java-version: adopt@1.11 | ||
| - uses: actions/cache@v2 | ||
| with: | ||
| path: ~/.cache | ||
| key: ${{ runner.os }}-jdk11-${{ hashFiles('**/*.sbt') }} | ||
| restore-keys: ${{ runner.os }}-jdk11- | ||
|
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Is this key correct?
Member
Author
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Yes. It has fallback mechanism https://github.com/actions/cache#inputs (fixed the link) There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I read below document after I wrote above comment. And though I understood the specification of |
||
| - name: Test | ||
| run: ./sbt test | ||
| - name: Universal Buffer Test | ||
| run: ./sbt test -J-Dmsgpack.universal-buffer=true | ||
| test_jdk8: | ||
| name: Test JDK8 | ||
| runs-on: ubuntu-latest | ||
| steps: | ||
| - uses: actions/checkout@v2 | ||
| - uses: olafurpg/setup-scala@v7 | ||
| with: | ||
| java-version: adopt@1.8 | ||
| - uses: actions/cache@v2 | ||
| with: | ||
| path: ~/.cache | ||
| key: ${{ runner.os }}-jdk8-${{ hashFiles('**/*.sbt') }} | ||
| restore-keys: ${{ runner.os }}-jdk8- | ||
| - name: Test | ||
| run: ./sbt test | ||
| - name: Universal Buffer Test | ||
| run: ./sbt test -J-Dmsgpack.universal-buffer=true | ||
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If the AdoptJDK installation is supported in setup-java action, it's better to replace with it.
https://github.com/actions/setup-java
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
msgpack-java's test is written in Scala, and also setup-scala can install natively-built sbt (csbt) using GraalVM. So let me keep using setup-scala.