Skip to content

Make clippy-runtime pass - #431

Merged
ifdotpy merged 1 commit into
mainfrom
fix/clippy-runtime
Oct 22, 2026
Merged

ifdotpy merged 1 commit into
mainfrom
fix/clippy-runtime

Conversation

@ifdotpy

@ifdotpy ifdotpy commented Oct 22, 2026 •

Copy link
Copy Markdown
Owner

Bug fix, no behavior change (stabilization freeze).

After rebase on main (which already boxes the large preparation futures), only test-side fixes remain:

  • Allow clippy::unwrap_used in the thread_prepare test module (same pattern as other runtime-services test modules).
  • Linux-only test imports (process_support as support in wave7_p01_factory.rs and provider_session_task.rs) are now cfg(target_os = "linux").
  • Remove the unused host_runtime re-export in runtime-server tests/integration.rs.

Results (rebased on origin/main):

  • just clippy-runtime: pass (local needs an untracked web/dist/index.html for the Tauri crate)
  • cargo nextest run -p studio-runtime-services -p studio-runtime-process (ci profile): 2112 passed, 8 skipped
  • just test-fast: passed before the rebase (750 s); the rebased diff is test-only lint changes

@ifdotpy
ifdotpy force-pushed the fix/clippy-runtime branch from 11b8164 to 7c58483 Compare October 22, 2026 12:57
@ifdotpy
ifdotpy merged commit 23fc606 into main Oct 22, 2026
5 of 8 checks passed
nowaalex pushed a commit that referenced this pull request Oct 22, 2026
…ts need

#431 removed it, but runtime_shutdown.rs (included by the integration test
target) imports crate::host_runtime, so the target stopped compiling.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
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