Skip to content
Open
Show file tree
Hide file tree
Changes from all commits
Commits
File filter

Filter by extension

Filter by extension


Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
13 changes: 4 additions & 9 deletions .github/workflows/build-rust-binding.yml
Original file line number Diff line number Diff line change
Expand Up @@ -14,29 +14,24 @@ jobs:
strategy:
fail-fast: false
matrix:
node-version: [18.x]
node-version: [22]

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

⚠️ Potential issue | 🔴 Critical

PR 目标与代码不匹配:缺少 Node.js 24 版本。

PR 标题为"chore: node24",PR 目标明确提到"为 Node.js 22、Node.js 24 和 macOS ARM 架构添加测试",但矩阵中仅包含 [22],缺少 Node.js 24。根据最新信息,Node.js 24 已于 2025 年 10 月 31 日正式进入长期支持(LTS)并将继续接收维护和安全更新至 2028 年 4 月。

请确认是否应在矩阵中添加 Node.js 24。如果是有意为之,请更新 PR 描述以反映实际的测试范围。

建议的修复:

-        node-version: [22]
+        node-version: [22, 24]
📝 Committable suggestion

‼️ IMPORTANT
Carefully review the code before committing. Ensure that it accurately replaces the highlighted code, contains no missing lines, and has no issues with indentation. Thoroughly test & benchmark the code to ensure it meets the requirements.

Suggested change
node-version: [22]
node-version: [22, 24]
🤖 Prompt for AI Agents
.github/workflows/build-rust-binding.yml around line 17: the GitHub Actions
node-version matrix currently lists only [22] while the PR title/description
claims tests for Node.js 22 and 24; update the matrix to include Node.js 24
(e.g., node-version: [22, 24]) so CI runs for both versions, or if 24 is
intentionally omitted, adjust the PR title/description to accurately reflect the
actual tested versions.

settings:
- host: macos-13
target: x86_64-apple-darwin
build: |
pnpm build:binding:release
strip -x crates/native_binding/*.node
- host: windows-latest
build: pnpm build:binding:release
target: x86_64-pc-windows-msvc
- host: ubuntu-22.04
- host: ubuntu-latest
target: x86_64-unknown-linux-gnu
docker: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-debian
build: |-
set -e &&
npm install -g corepack@0.31.0 &&
pnpm build:binding:release --target x86_64-unknown-linux-gnu &&
strip crates/native_binding/*.node
- host: ubuntu-22.04
- host: ubuntu-latest
target: x86_64-unknown-linux-musl
docker: ghcr.io/napi-rs/napi-rs/nodejs-rust:lts-alpine
build: set -e && pnpm build:binding:release && strip crates/native_binding/*.node
- host: macos-13
- host: macos-latest
target: aarch64-apple-darwin
build: |
pnpm build:binding:release --target aarch64-apple-darwin
Expand Down
18 changes: 7 additions & 11 deletions .github/workflows/nodejs.yml
Original file line number Diff line number Diff line change
Expand Up @@ -39,20 +39,16 @@ jobs:
- build-rust-wasm
strategy:
matrix:
node-version: [18.x, 20.x]
host: [macos-13, windows-latest, ubuntu-latest]
node-version: [22, 20]
host: [macos-latest, windows-latest, ubuntu-latest]
exclude:
- node-version: 18.x
host: macos-13
- node-version: 18.x
host: windows-latest
- node-version: 20.x
host: macos-13
- node-version: 20.x
- node-version: 20
host: macos-latest
- node-version: 20
host: windows-latest
include:
- host: macos-13
target: x86_64-apple-darwin
- host: macos-latest
target: aarch64-apple-darwin
- host: windows-latest
target: x86_64-pc-windows-msvc
- host: ubuntu-latest
Expand Down
8 changes: 4 additions & 4 deletions .github/workflows/publish.yml
Original file line number Diff line number Diff line change
Expand Up @@ -98,10 +98,10 @@ jobs:
uses: pnpm/action-setup@v4
with:
version: 10
- name: Setup Node 18
- name: Setup Node
uses: actions/setup-node@v6
with:
node-version: 18
node-version: 22
cache: 'pnpm'
registry-url: 'https://registry.npmjs.org' # Don't touch!
- name: Git Identity
Expand Down Expand Up @@ -168,14 +168,14 @@ jobs:

# Bootstrap project
- name: Install dependencies
run: pnpm -r install --frozen-lockfile
run: pnpm install --frozen-lockfile
- name: Download all artifacts
uses: actions/download-artifact@v6
with:
path: crates/native_binding/artifacts
- name: Move artifacts
run: pnpm artifacts
- name: Checkout Harmony Commond Line Tools
- name: Checkout Harmony Command Line Tools
env:
COMMANDLINE_TOOLS_URL: https://storage.360buyimg.com/taro-patch/command-line-tools-for-harmony/5.0.13.200/linux_x64
TOOLS_HOME: ${{ runner.os == 'macOS' && '~/harmony-tools' || '$HOME/harmony-tools' }}
Expand Down
4 changes: 2 additions & 2 deletions .github/workflows/sync-components-types.yml
Original file line number Diff line number Diff line change
Expand Up @@ -42,10 +42,10 @@ jobs:
args: [--frozen-lockfile, --strict-peer-dependencies]
- args: [--filter @tarojs/components, -D, miniapp-types@latest]
# Note: 当前同步脚本使用 ts-node 与 node20 存在兼容问题,修复后解除版本限制
- name: Setup Node 18
- name: Setup Node
uses: actions/setup-node@v6
with:
node-version: 18
node-version: 22
cache: 'pnpm'
registry-url: 'https://registry.npmjs.org' # Don't touch!

Expand Down
1 change: 1 addition & 0 deletions packages/taro-components/scripts/stencil/stencil.config.ts
Original file line number Diff line number Diff line change
Expand Up @@ -98,6 +98,7 @@ export const config: Config = {
* 实际并不支持 clone 模式,暂不考虑修复
*/
testing: {
browserArgs: ['--no-sandbox', '--disable-setuid-sandbox'],
globals: {
'ts-jest': {
diagnostics: false,
Expand Down