Skip to content

feat(ci): enforce Rust 1.80 MSRV in Cargo manifests and GitHub Actions (Closes #650) - #758

Open
stayzappy wants to merge 2 commits into
Nanle-code:masterfrom
stayzappy:feature/issue-650-enforce-msrv-1.80
Open

feat(ci): enforce Rust 1.80 MSRV in Cargo manifests and GitHub Actions (Closes #650)#758
stayzappy wants to merge 2 commits into
Nanle-code:masterfrom
stayzappy:feature/issue-650-enforce-msrv-1.80

Conversation

@stayzappy

Copy link
Copy Markdown

Summary

Closes #650

Key Changes Made

  1. Declared Rust 1.80 MSRV:
    • Added rust-version = "1.80" to Cargo.toml, crates/starforge-plugin-sdk/Cargo.toml, and crates/starforge-wasm/Cargo.toml.
  2. Configured GitHub Actions MSRV CI Job:
    • Added msrv verification job to .github/workflows/ci.yml using dtolnay/rust-toolchain@1.80.0 running cargo check --locked --workspace.
  3. Workspace MSRV Compatibility Fixes:
    • Updated Migration trait method signatures in src/utils/database.rs (&mut Connection -> &Connection).
    • Exported missing modules and functions (pub mod ai_doc_qa, pub fn read_spec_entries).
    • Derived PartialEq, Eq on ComplianceSeverity.
    • Fixed test struct initializers across the workspace.

Verification

  • cargo check --workspace passes cleanly with zero compilation errors.

@drips-wave

drips-wave Bot commented Aug 25, 2026

Copy link
Copy Markdown

@stayzappy Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

[2026 Hardening] Enforce the Rust 1.80 MSRV in CI

1 participant