refactor(ci): move SODIUM_LIB_DIR to job-level env
Some checks failed
PR Review Automation / review (pull_request) Successful in 3m20s
Test / rust-clippy (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Test / frontend-typecheck (pull_request) Has been cancelled
Test / frontend-tests (pull_request) Has been cancelled
Test / rust-tests (pull_request) Has been cancelled
Some checks failed
PR Review Automation / review (pull_request) Successful in 3m20s
Test / rust-clippy (pull_request) Has been cancelled
Test / rust-fmt-check (pull_request) Has been cancelled
Test / frontend-typecheck (pull_request) Has been cancelled
Test / frontend-tests (pull_request) Has been cancelled
Test / rust-tests (pull_request) Has been cancelled
Moved SODIUM_LIB_DIR from per-step env blocks to job-level env for all three Rust test jobs (rust-fmt-check, rust-clippy, rust-tests). Benefits: - Applies to ALL cargo commands in the job, including generate-lockfile - More maintainable - single declaration per job - Consistent with best practices for job-wide environment variables Addresses automated review feedback. Co-Authored-By: Claude Sonnet 4.5 <noreply@anthropic.com>
This commit is contained in:
parent
b20deab391
commit
1172f20137
@ -12,6 +12,8 @@ jobs:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: rustlang/rust:nightly
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
steps:
|
||||
- name: Checkout
|
||||
run: |
|
||||
@ -63,16 +65,14 @@ jobs:
|
||||
- name: Update version from Git
|
||||
run: node scripts/update-version.mjs
|
||||
- run: cargo generate-lockfile --manifest-path src-tauri/Cargo.toml
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
- run: cargo fmt --manifest-path src-tauri/Cargo.toml --check
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
|
||||
rust-clippy:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: rustlang/rust:nightly
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
steps:
|
||||
- name: Checkout
|
||||
run: |
|
||||
@ -106,13 +106,13 @@ jobs:
|
||||
- name: Install clippy
|
||||
run: rustup component add clippy
|
||||
- run: cargo clippy --manifest-path src-tauri/Cargo.toml -- -D warnings
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
|
||||
rust-tests:
|
||||
runs-on: ubuntu-latest
|
||||
container:
|
||||
image: rustlang/rust:nightly
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
steps:
|
||||
- name: Checkout
|
||||
run: |
|
||||
@ -144,13 +144,9 @@ jobs:
|
||||
libsodium-dev \
|
||||
pkg-config
|
||||
- run: cargo test --manifest-path src-tauri/Cargo.toml -- --test-threads=1
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
|
||||
- name: Run shell module tests
|
||||
run: 'cargo test --manifest-path src-tauri/Cargo.toml "shell::" -- --test-threads=1'
|
||||
env:
|
||||
SODIUM_LIB_DIR: /usr/lib/x86_64-linux-gnu
|
||||
|
||||
frontend-typecheck:
|
||||
runs-on: ubuntu-latest
|
||||
|
||||
Loading…
Reference in New Issue
Block a user