Skip to content

Commit

Permalink
feat(isthumus): extract CLI into isthmus-cli project #248 (#249)
Browse files Browse the repository at this point in the history
BREAKING CHANGE: CLI related functionality is now in the io.substrait.isthmus.cli package
  • Loading branch information
bestbeforetoday authored May 2, 2024
1 parent 72bab63 commit a49de62
Show file tree
Hide file tree
Showing 16 changed files with 571 additions and 513 deletions.
9 changes: 5 additions & 4 deletions .github/workflows/pr.yml
Original file line number Diff line number Diff line change
Expand Up @@ -71,17 +71,18 @@ jobs:
- name: Build with Gradle
run: gradle nativeImage
- name: Smoke Test
run: ./isthmus/src/test/script/smoke.sh
./isthmus/src/test/script/tpch_smoke.sh
run: |
./isthmus-cli/src/test/script/smoke.sh
./isthmus-cli/src/test/script/tpch_smoke.sh
- name: Rename the artifact to OS-unique name
shell: bash
run: |
value=`mv isthmus/build/graal/isthmus isthmus/build/graal/isthmus-${{ matrix.os }}`
value=`mv isthmus-cli/build/graal/isthmus isthmus-cli/build/graal/isthmus-${{ matrix.os }}`
- name: Publish artifact
uses: actions/upload-artifact@v4
with:
name: isthmus-${{ matrix.os }}
path: isthmus/build/graal/isthmus-${{ matrix.os }}
path: isthmus-cli/build/graal/isthmus-${{ matrix.os }}
dry-run-release:
name: Dry-run release
runs-on: ubuntu-latest
Expand Down
9 changes: 5 additions & 4 deletions .github/workflows/release.yml
Original file line number Diff line number Diff line change
Expand Up @@ -37,17 +37,18 @@ jobs:
- name: Build with Gradle
run: gradle nativeImage
- name: Smoke Test
run: ./isthmus/src/test/script/smoke.sh
./isthmus/src/test/script/tpch_smoke.sh
run: |
./isthmus-cli/src/test/script/smoke.sh
./isthmus-cli/src/test/script/tpch_smoke.sh
- name: Rename the artifact to OS-unique name
shell: bash
run: |
value=`mv isthmus/build/graal/isthmus isthmus/build/graal/isthmus-${{ matrix.os }}`
value=`mv isthmus-cli/build/graal/isthmus isthmus-cli/build/graal/isthmus-${{ matrix.os }}`
- name: Publish artifact
uses: actions/upload-artifact@v4
with:
name: isthmus-${{ matrix.os }}
path: isthmus/build/graal/isthmus-${{ matrix.os }}
path: isthmus-cli/build/graal/isthmus-${{ matrix.os }}
semantic-release:
if: github.repository == 'substrait-io/substrait-java'
runs-on: ubuntu-latest
Expand Down
2 changes: 1 addition & 1 deletion gradle.properties
Original file line number Diff line number Diff line change
Expand Up @@ -20,7 +20,7 @@ guava.version=32.1.3-jre
immutables.version=2.10.1
jackson.version=2.16.1
junit.version=5.8.1
protobuf.version=3.17.1
protobuf.version=3.17.3
slf4j.version=2.0.13

#version that is going to be updated automatically by releases
Expand Down
Loading

0 comments on commit a49de62

Please sign in to comment.