Skip to content

chore: fix clippy 1.99 fallout - #140

Merged
akainth015 merged 1 commit into
mainfrom
pr/clippy-199
Oct 16, 2026
Merged

akainth015 merged 1 commit into
mainfrom
pr/clippy-199

Conversation

@akainth015

@akainth015 akainth015 commented Oct 16, 2026 •

Copy link
Copy Markdown
Member

Allow clippy::double_must_use workspace-wide (#[async_trait] expands trait methods to boxed futures tagged #[must_use], tripping the new clippy 1.99 lint on every such trait) and unroll a single-element for loop in infinity-daemon/build.rs.

Unblocks ./check.bash on current toolchains. First PR of a 6-PR stack.

📚 Stack (review bottom-up, merge top-down)

PR step
➡️ #140 1/6
#141 2/6
#142 3/6
#143 4/6
#144 5/6
#145 6/6

Base of the stack (targets main).
Merge #140 first; GitHub retargets each child PR's base automatically as parents land.

…e-element loop)

* Allow `clippy::double_must_use` workspace-wide: `#[async_trait]` expands
  trait methods to return boxed futures tagged `#[must_use]`, tripping the
  lint (new in clippy 1.99) on every such trait.
* Unroll a single-element `for` loop in `infinity-daemon/build.rs`.
@cloudflare-workers-and-pages

Copy link
Copy Markdown

Deploying infinity with  Cloudflare Pages  Cloudflare Pages

Latest commit: 2e4b04d
Status: ✅  Deploy successful!
Preview URL: https://362a2c0f.infinity-dc7.pages.dev
Branch Preview URL: https://pr-clippy-199.infinity-dc7.pages.dev

View logs

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