Open
Conversation
Contributor
JonatanWaern
commented
Mar 10, 2026
- Update subprocess to 1.0
- Clear some clippy warnings
- Use rust 1.93
Signed-off-by: Jonatan Waern <jonatan.waern@intel.com>
Signed-off-by: Jonatan Waern <jonatan.waern@intel.com>
Signed-off-by: Jonatan Waern <jonatan.waern@intel.com>
There was a problem hiding this comment.
Pull request overview
This PR updates the codebase to build cleanly on Rust 1.93 by adopting newer Option/Result helper APIs, upgrading subprocess to 1.0, and adjusting CI to target Rust 1.93.
Changes:
- Replace many
map_or(...)patterns withis_some_and/is_ok_and/is_none_orto address clippy warnings and modernize idioms. - Upgrade
subprocessfrom=0.2.9to1.0and migrate DFA client process spawning to the new API (Exec/Job). - Update GitHub Actions workflows to use Rust 1.93.
Reviewed changes
Copilot reviewed 22 out of 22 changed files in this pull request and generated 1 comment.
Show a summary per file
| File | Description |
|---|---|
| src/server/mod.rs | Uses is_ok_and / is_some_and to modernize checks and reduce clippy noise. |
| src/main.rs | Removes unused clap imports while keeping derive helper attributes. |
| src/lsp_data.rs | Updates URI scheme checks and capability checks using is_some_and. |
| src/dfa/main.rs | Adapts error-code handling to the updated subprocess exit status API. |
| src/dfa/client.rs | Migrates subprocess spawning from Popen to Exec/Job for subprocess 1.0. |
| src/analysis/templating/topology.rs | Refactors map_or usage to is_some_and. |
| src/analysis/templating/objects.rs | Uses is_none_or; adds clippy allow for large_enum_variant. |
| src/analysis/symbols.rs | Adds clippy allow for large_enum_variant. |
| src/analysis/structure/objects.rs | Refactors optional token kind checks via is_some_and. |
| src/analysis/parsing/types.rs | Refactors parser lookahead loops to is_some_and. |
| src/analysis/parsing/structure.rs | Refactors optional token checks/loops to is_some_and. |
| src/analysis/parsing/statement.rs | Refactors loop continuation logic and adds clippy allow for large_enum_variant. |
| src/analysis/parsing/misc.rs | Refactors “empty list” detection using is_some_and. |
| src/analysis/parsing/expression.rs | Refactors expression parsing lookahead checks via is_some_and. |
| src/analysis/mod.rs | Refactors shallow/object kind checks and CanonPath comparisons using new helpers. |
| src/actions/semantic_lookup.rs | Refactors template-implementation presence check to is_some_and. |
| src/actions/notifications.rs | Refactors JSON object emptiness check via is_some_and. |
| src/actions/mod.rs | Refactors workspace path check and file watch comparison to is_ok_and / is_some_and. |
| src/actions/analysis_storage.rs | Refactors many Option/Result defaulting checks via is_none_or/is_ok_and/is_some_and. |
| Cargo.toml | Bumps subprocess dependency to 1.0. |
| .github/workflows/scans.yml | Pins Rust version to 1.93.0 and attempts to switch toolchain during runs. |
| .github/workflows/binaries.yml | Pins Rust version to 1.93.0 for builds/tests/clippy in CI. |
Comments suppressed due to low confidence (1)
.github/workflows/binaries.yml:35
rust_versionis updated, but the workflow later uses it withrustup default ${{ env.rust_version }}without ensuring that toolchain is installed. This can break CI on runners that don’t already have that exact version. Add an explicit toolchain install step (or a Rust toolchain setup action) before setting the default.
rust_version: 1.93.0
jobs:
build:
runs-on: ${{ inputs.os }}
steps:
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
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
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.