Skip to content

Conversation

@imor
Copy link
Contributor

@imor imor commented Jan 23, 2026

This PR bumps the Rust edition to 2024 and fixes clippy and formatting errors.

Copilot AI review requested due to automatic review settings January 23, 2026 08:41
Copy link
Contributor

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 bumps the Rust edition from 2021 to 2024 and applies automated clippy/formatting fixes to comply with the new edition requirements.

Changes:

  • Updated Rust edition to 2024 in all Cargo.toml files
  • Applied import reordering (alphabetical sorting) throughout the codebase
  • Converted nested if let statements to let-chains (if let && let) - a Rust 2024 feature
  • Added explicit unsafe {} blocks within unsafe fn functions per Rust 2024 requirements
  • Applied formatting fixes including trailing commas, semicolons, and alignment improvements
  • Wrapped env::set_var calls in unsafe {} blocks as required by Rust 2024

Reviewed changes

Copilot reviewed 51 out of 51 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
Cargo.toml Updated workspace edition to 2024 and rust-version to 1.88.0
wasm-wrappers/fdw/Cargo.toml Updated workspace edition to 2024 and rust-version to 1.88
wasm-wrappers/fdw/infura_fdw/Cargo.toml Updated edition to 2024
supabase-wrappers/src/*.rs Added redundant unsafe {} blocks, let-chains, import reordering
wrappers/src/fdw/**/*.rs Import reordering, let-chains, formatting, unsafe blocks for env::set_var
wrappers/src/fdw/auth0_fdw/auth0_client/row.rs Commented out unused Success struct
wrappers/src/stats.rs Import reordering
supabase-wrappers-macros/src/lib.rs Import reordering

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

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.

2 participants