forked from solana-labs/solana
-
Notifications
You must be signed in to change notification settings - Fork 331
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
New clippy lints in solana-sbf-rust-invoke #4479
Comments
@pgarg66 Here's another one. I'm not sure what the right fix is, so it would be helpful to have someone on your team handle this correctly. Thanks in advance! |
I'll have a look! |
I simply switched to static in #4503. I explained the reason in the PR's summary. Let's wait for the CI to turn green now 🤞 |
Closing the issue, since #4503 was merged. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
There are new clippy lints when upgrade nightly rust to 2024-11-22 (1.84.0), which causes CI to fail:
I used this cmd to reproduce locally, from within
programs/sbf/
:Should this array be static? Or a slice? Or add a clippy-allow? Or change how the assert is checked (e.g. use an iter that always returns
0
s)?Child of #4380
The text was updated successfully, but these errors were encountered: