Skip to content

Fix clippy linting errors in turbopack-dev-server and pack-napi#2575

Closed
Copilot wants to merge 8 commits intonextfrom
copilot/fix-clippy-linting-errors
Closed

Fix clippy linting errors in turbopack-dev-server and pack-napi#2575
Copilot wants to merge 8 commits intonextfrom
copilot/fix-clippy-linting-errors

Conversation

Copy link

Copilot AI commented Feb 4, 2026

Fix Clippy Linting Errors

  • Fix duplicate import of OperationVc in turbopack-dev-server/src/lib.rs
  • Fix conditional compilation issues in pack-napi/src/util.rs DhatProfilerGuard::try_init()
  • Apply proper cargo fmt formatting
  • Update next.js submodule with clippy fix
  • Run clippy to verify all errors are fixed
  • Run cargo fmt to verify formatting compliance
Original prompt

Fix all clippy linting errors in this pull request to ensure compliance with Rust best practices.

This pull request was created from Copilot chat.


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

Co-authored-by: xusd320 <16577489+xusd320@users.noreply.github.com>
@xusd320
Copy link
Contributor

xusd320 commented Feb 4, 2026

@copilot cargo fmt failed

Co-authored-by: xusd320 <16577489+xusd320@users.noreply.github.com>
Copilot AI changed the title [WIP] Fix clippy linting errors for Rust compliance Fix clippy linting errors in turbopack-dev-server and pack-napi Feb 4, 2026
Copilot AI requested a review from xusd320 February 4, 2026 08:30
Copy link
Author

Copilot AI commented Feb 4, 2026

@copilot cargo fmt failed

Fixed in commit 9247aae. Applied proper formatting to the #[cfg(all(...))] attribute to satisfy cargo fmt requirements.

@xusd320
Copy link
Contributor

xusd320 commented Feb 4, 2026

@copilot should update the submodule

Copy link
Author

Copilot AI commented Feb 4, 2026

@copilot should update the submodule

Updated the next.js submodule in commit 6e343c2. The submodule now points to commit d537811 which includes the fix for the duplicate OperationVc import in turbopack-dev-server.

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