Skip to content

Comments

chore: Fix Rust warning about unused variable#28544

Closed
maximiliankolb wants to merge 1 commit intodenoland:mainfrom
maximiliankolb:fix_warning
Closed

chore: Fix Rust warning about unused variable#28544
maximiliankolb wants to merge 1 commit intodenoland:mainfrom
maximiliankolb:fix_warning

Conversation

@maximiliankolb
Copy link

Unused variables cause warnings via "cargo build".

Fixes #28532

If GitHub Actions fail, I am kindly pinging/asking dsherret for help.

Unused variables cause warnings via "cargo build".

Fixes #28532
@CLAassistant
Copy link

CLA assistant check
Thank you for your submission! We really appreciate it. Like many open source projects, we ask that you sign our Contributor License Agreement before we can accept your contribution.
You have signed the CLA already but the status is still pending? Let us recheck it.

Copy link
Member

@dsherret dsherret left a comment

Choose a reason for hiding this comment

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

LGTM. Can you sign the CLA and then this should merge automatically.

@dsherret dsherret enabled auto-merge (squash) March 20, 2025 14:36
@dsherret
Copy link
Member

@maximiliankolb just a reminder about this one.

auto-merge was automatically disabled April 23, 2025 19:33

Pull request was closed

@maximiliankolb maximiliankolb deleted the fix_warning branch April 23, 2025 19:33
@maximiliankolb
Copy link
Author

I do not understand the CLA and I will not sign it. Feel free to cherry-pick my change from this upstream PR as per MIT license.

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.

Compiling Deno v2.2.4 causes warning for Deno Runtime v0.203.0

3 participants