Fix clippy error in webhook due to rust 1.91.0

[kumarUjjawal]

Pull Request

Related issue

Fixes #731

What does this PR do?

• Fix clippy issue introduced in webhook module.

PR checklist

Please check if your PR fulfills the following requirements:

• Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
• Have you read the contributing guidelines?
• Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

Summary by CodeRabbit

• Refactor
  • Optimized internal implementation of default value handling to align with Rust best practices.

[coderabbitai]

Walkthrough

The PR addresses a new Clippy lint (clippy::derivable_impls) introduced in Rust 1.91.0 by replacing the manual Default implementation for the HeadersUpdate enum with a derived implementation, using the #[default] attribute on the NotSet variant.

Changes

Cohort / File(s)	Summary
Default trait derivation src/webhooks.rs	Added Default to the derive macro for HeadersUpdate enum and marked NotSet variant with #[default] attribute. Removed the manual impl Default for HeadersUpdate block.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

• Straightforward refactoring from manual to derived implementation
• Functionally equivalent to previous code
• Single file and single enum affected
• Addresses a Clippy compliance issue with a standard pattern

Poem

🐰 A rabbit hops through clippy's new refrain,
No boilerplate—derive shall reign!
#[default] marks the way,
Less code to write, more time to play! 🎉

Pre-merge checks and finishing touches

✅ Passed checks (5 passed)

Check name	Status	Explanation
Description Check	✅ Passed	Check skipped - CodeRabbit’s high-level summary is enabled.
Title check	✅ Passed	The title clearly and specifically summarizes the main change: fixing a clippy error in the webhook module triggered by Rust 1.91.0, which matches the changeset.
Linked Issues check	✅ Passed	The PR successfully addresses issue #731 by applying the derivable_impls fix: using derive Default instead of manual impl for HeadersUpdate, making the clippy lint pass.
Out of Scope Changes check	✅ Passed	All changes are scoped to fixing the clippy error in src/webhooks.rs; no unrelated modifications to other modules or functionality are present.
Docstring Coverage	✅ Passed	No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.

✨ Finishing touches

• 📝 Generate docstrings

🧪 Generate unit tests (beta)

• Create PR with unit tests
• Post copyable unit tests in a comment

---

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between e90a3c2 and 0786d90.

📒 Files selected for processing (1)

• src/webhooks.rs (1 hunks)

⏰ Context from checks skipped due to timeout of 90000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (1)

• GitHub Check: integration-tests

🔇 Additional comments (1)

src/webhooks.rs (1)

69-75: LGTM! Clippy lint correctly resolved.

The change from a manual Default implementation to a derived one using #[default] on the NotSet variant correctly addresses the clippy::derivable_impls lint introduced in Rust 1.91.0. The behavior is preserved, and since HeadersUpdate is a private enum, there are no public API implications.

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

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

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 86.18%. Comparing base (e90a3c2) to head (0786d90).
⚠️ Report is 2 commits behind head on main.

Additional details and impacted files

@@            Coverage Diff             @@
##             main     #732      +/-   ##
==========================================
- Coverage   86.18%   86.18%   -0.01%     
==========================================
  Files          20       20              
  Lines        6263     6260       -3     
==========================================
- Hits         5398     5395       -3     
  Misses        865      865              

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[curquiza]

Nice, thanks a lot

[curquiza]

bors merge

[meili-bors]

Build succeeded:

• rust-format
• readme-check
• wasm-build
• Yaml linting check
• integration-tests
• clippy-check