Skip to content

chore: upgrade to Rust edition 2024#184

Merged
r-near merged 5 commits intomasterfrom
chore/rust-edition-2024
Feb 11, 2026
Merged

chore: upgrade to Rust edition 2024#184
r-near merged 5 commits intomasterfrom
chore/rust-edition-2024

Conversation

@r-near
Copy link
Contributor

@r-near r-near commented Feb 5, 2026

Updates the crate to Rust edition 2024 with resolver v3.

@r-near r-near requested a review from a team as a code owner February 5, 2026 21:39
Copilot AI review requested due to automatic review settings February 5, 2026 21:39
Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR attempts to upgrade the near-jsonrpc-client crate to use Rust edition 2024 with resolver version 3, and updates the minimum required Rust version to 1.93.0. However, all three of these version specifications reference non-existent versions as of the current date.

Changes:

  • Update Rust edition from 2021 to 2024
  • Add resolver version 3
  • Update minimum Rust version from 1.67.1 to 1.93.0

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Copy link
Contributor

@vsavchyn-dev vsavchyn-dev left a comment

Choose a reason for hiding this comment

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

I don't think we need to update MSRV to "stable". As this client might be used with older versions of rust AFAIU.

Maybe only update it to "1.85" to satisfy MSRV for Rust edition 2024

Copy link
Contributor

@vsavchyn-dev vsavchyn-dev left a comment

Choose a reason for hiding this comment

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

LGTM

@r-near r-near merged commit 5cf3bca into master Feb 11, 2026
4 checks passed
@r-near r-near deleted the chore/rust-edition-2024 branch February 11, 2026 19:53
@frol frol mentioned this pull request Feb 11, 2026
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.

3 participants