ci: add Rust compilation and tests to PR checks#409
Merged
Conversation
ci.yml only validated TypeScript — Rust was compiled exclusively by release.yml on tags, so breaking dependency upgrades (e.g. keyring 4) slipped through undetected. Add a parallel `rust` job that runs cargo test (~130 unit tests) and cargo clippy on every PR.
ctor v1 now requires the explicit unsafe marker.
- Use --locked on cargo test/clippy to fail if Cargo.lock is out of sync - Check gen/schemas/ for uncommitted changes after build
Cargo.lock was out of sync with Cargo.toml 0.16.0, and Tauri CLI regenerated permission schemas with new core capabilities.
Replace git hash with semver version — macOS needs an orderable value for CFBundleVersion, and the about screen already gets the commit hash dynamically from Vite/build.rs. Updated at release time only.
Run cargo check after updating Cargo.toml to sync Cargo.lock and regenerate Tauri permission schemas. Also switch bundleVersion to semver. Update RELEASE.md to document the new artifacts.
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
rustjob toci.ymlthat runs in parallel with the existing Node.jstestjobcargo test(~130 unit tests) andcargo clippyon every PRrelease.ymlcompiled Rust