Skip to content

Conversation

Copy link

Copilot AI commented Dec 30, 2025

CI was failing due to new Rust 1.92.0 clippy lints triggered by the CVE dependency updates.

Changes

  • Replace manual modulo checks with is_multiple_of(): Updated 4 locations in src/main.rs and src/neo4j_client.rs where count % N == 0 was used for progress logging
  • Modernize error construction: Replaced io::Error::new(std::io::ErrorKind::Other, msg) with io::Error::other(msg) in src/utils.rs
// Before
if counter % 1000 == 0 {
    info!("processed {} queries", counter);
}

// After
if counter.is_multiple_of(1000) {
    info!("processed {} queries", counter);
}

These are idiomatic improvements with no functional changes.


✨ Let Copilot coding agent set things up for you — coding agent works faster and does higher quality work when set up for your repo.

@vercel
Copy link

vercel bot commented Dec 30, 2025

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Review Updated (UTC)
benchmark Ready Ready Preview, Comment Dec 30, 2025 2:35pm

@coderabbitai
Copy link

coderabbitai bot commented Dec 30, 2025

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.


Comment @coderabbitai help to get the list of available commands and usage tips.

Co-authored-by: gkorland <753206+gkorland@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix React Server Components CVE vulnerabilities Fix Rust clippy lints for manual is_multiple_of and io::Error::other Dec 30, 2025
Copilot AI requested a review from gkorland December 30, 2025 14:37
@gkorland gkorland marked this pull request as ready for review December 30, 2025 15:59
@gkorland gkorland merged commit 9077621 into vercel/react-server-components-cve-vu-gjkikg Dec 30, 2025
3 checks passed
@gkorland gkorland deleted the copilot/sub-pr-69 branch December 30, 2025 16:00
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