Skip to content

Commit

Permalink
Merge pull request #492 from pelias/remove-jdk-configuration
Browse files Browse the repository at this point in the history
Remove JDK configuration from CI
  • Loading branch information
orangejulius authored Feb 11, 2025
2 parents 1319c20 + 45bec95 commit 4d6d74d
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 6 deletions.
4 changes: 1 addition & 3 deletions .github/workflows/_integration_tests.yml
Original file line number Diff line number Diff line change
Expand Up @@ -9,18 +9,16 @@ jobs:
- ubuntu-22.04
node-version: [18.x, 20.x, 22.x]
es-version: [7.6.1]
jdk-version: [oraclejdk11]
icuTokenizer: [true, false]
steps:
- uses: actions/checkout@v4
- name: Install node.js ${{ matrix.node-version }}
uses: actions/setup-node@v4
with:
node-version: ${{ matrix.node-version }}
- name: Start elasticsearch ${{ matrix.es-version }} (${{ matrix.jdk-version }})
- name: Start elasticsearch ${{ matrix.es-version }}
env:
ES_VERSION: ${{ matrix.es-version }}
JDK_VERSION: ${{ matrix.jdk-version }}
run: ./scripts/setup_ci.sh
- name: Run integration tests
run: |
Expand Down
3 changes: 0 additions & 3 deletions scripts/setup_ci.sh
Original file line number Diff line number Diff line change
Expand Up @@ -4,9 +4,6 @@ set -e
# create elasticsearch directory
mkdir /tmp/elasticsearch

# allow switching the JDK version
curl -s https://raw.githubusercontent.com/michaelklishin/jdk_switcher/master/jdk_switcher.sh | bash -s use "${JDK_VERSION}"

# download and install elasticsearch with ICU plugin
FILENAME="elasticsearch-${ES_VERSION}-linux-x86_64.tar.gz"
STRIP_COMPONENTS=1
Expand Down

0 comments on commit 4d6d74d

Please sign in to comment.