[KYUUBI #6712] [UI] Bump pnpm to v9 and nodejs to v20 LTS
# 🔍 Description ## Issue References 🔗 This pull request fixes # ## Describe Your Solution 🔧 - bump `pnpm` to v9.11, release note: https://github.com/pnpm/pnpm/releases/tag/v9.11.0 - updated pnpm lockfile by running `pnpm install --fix-lockfile` - bump `nodejs` version from v18 to v20 (LTS version, released on 18 Apr 2023) - update related github actions, aligning the pnpm and nodejs versions ## Types of changes 🔖 - [ ] Bugfix (non-breaking change which fixes an issue) - [ ] New feature (non-breaking change which adds functionality) - [ ] Breaking change (fix or feature that would cause existing functionality to change) ## Test Plan 🧪 #### Behavior Without This Pull Request ⚰️ #### Behavior With This Pull Request 🎉 #### Related Unit Tests --- # Checklist 📝 - [ ] This patch was not authored or co-authored using [Generative Tooling](https://www.apache.org/legal/generative-tooling.html) **Be nice. Be informative.** Closes #6712 from bowenliang123/pnpm9. Closes #6712 ca6e425f4 [liangbowen] nit a9ea92216 [liangbowen] bump pnpm to v9 Lead-authored-by: Bowen Liang <liangbowen@gf.com.cn> Co-authored-by: liangbowen <liangbowen@gf.com.cn> Signed-off-by: liangbowen <liangbowen@gf.com.cn>
This commit is contained in:
parent
ebe7e922ee
commit
9ef38b9582
12
.github/workflows/style.yml
vendored
12
.github/workflows/style.yml
vendored
@ -88,15 +88,21 @@ jobs:
|
||||
SPOTLESS_BLACK_VERSION=$(build/mvn help:evaluate -Dexpression=spotless.python.black.version -q -DforceStdout)
|
||||
pip install black==$SPOTLESS_BLACK_VERSION
|
||||
build/mvn spotless:check ${{ matrix.profiles }} -Pspotless-python
|
||||
- name: Get NodeJS and PNPM version
|
||||
run: |
|
||||
NODEJS_VERSION=$(build/mvn help:evaluate -Dexpression=node.version -q -DforceStdout)
|
||||
PNPM_VERSION=$(build/mvn help:evaluate -Dexpression=pnpm.version -q -DforceStdout)
|
||||
echo "NODEJS_VERSION=${NODEJS_VERSION}" >> "$GITHUB_ENV"
|
||||
echo "PNPM_VERSION=${PNPM_VERSION}" >> "$GITHUB_ENV"
|
||||
- name: setup npm
|
||||
uses: actions/setup-node@v4
|
||||
with:
|
||||
node-version: 18
|
||||
node-version: ${{env.NODEJS_VERSION}}
|
||||
- name: Web UI Style with node
|
||||
run: |
|
||||
cd ./kyuubi-server/web-ui
|
||||
npm install pnpm@8 -g
|
||||
pnpm install
|
||||
npm install pnpm@${PNPM_VERSION} -g
|
||||
pnpm install --frozen-lockfile
|
||||
pnpm run lint
|
||||
echo "---------------------------------------Notice------------------------------------"
|
||||
echo "If this style check fail, you should follow these commands for a tentative fix"
|
||||
|
||||
2
.github/workflows/web-ui.yml
vendored
2
.github/workflows/web-ui.yml
vendored
@ -50,6 +50,6 @@ jobs:
|
||||
run: |
|
||||
cd ./kyuubi-server/web-ui
|
||||
npm install pnpm@${PNPM_VERSION} -g
|
||||
pnpm install
|
||||
pnpm install --frozen-lockfile
|
||||
pnpm run coverage
|
||||
pnpm run build
|
||||
|
||||
File diff suppressed because it is too large
Load Diff
Loading…
Reference in New Issue
Block a user