Skip to content

ci: run Rust tests in desktop gate - #15

Merged
fly1d merged 2 commits into
mainfrom
codex/run-rust-tests-in-ci
Aug 11, 2026
Merged

ci: run Rust tests in desktop gate#15
fly1d merged 2 commits into
mainfrom
codex/run-rust-tests-in-ci

Conversation

@fly1d

@fly1d fly1d commented Aug 11, 2026

Copy link
Copy Markdown
Owner

Summary

  • replace duplicated Rust commands in the Desktop workflow with npm run desktop:check
  • ensure the two Rust provider security tests execute on the macOS CI runner
  • keep local and CI desktop gates defined in one place

Verification

  • npm run desktop:check (2 Rust tests passed)
  • workflow YAML parsed successfully
  • diff check passed

@fly1d fly1d left a comment

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

Reviewed the complete diff. The desktop workflow now delegates to the shared desktop:check script, which adds the Rust test suite while retaining fmt, Clippy, and compilation checks. The workflow file is also included in both pull_request and push path filters. CI, browser smoke, CodeQL, and the macOS desktop gate all pass. No blocking findings.

@fly1d
fly1d merged commit eae30eb into main Aug 11, 2026
4 checks passed
@fly1d
fly1d deleted the codex/run-rust-tests-in-ci branch August 11, 2026 10:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant