Skip to content

Conversation

@erskingardner
Copy link
Member

@erskingardner erskingardner commented Oct 17, 2025

Summary by CodeRabbit

  • Chores
    • Updated internal dependencies to their latest compatible versions.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Oct 17, 2025

Walkthrough

Updated two git revision pins in rust/Cargo.toml: mdk-core from f9550e69... to f46875ec... and whitenoise from 063f7b27... to 474fbaec.... No changes to public declarations detected.

Changes

Cohort / File(s) Summary
Dependency version updates
rust/Cargo.toml
Updated git revision pins for mdk-core and whitenoise dependencies

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

Possibly related PRs

Suggested reviewers

  • jgmontoya
  • josefinalliende
  • codeswot

Poem

🐰 Hop, hop, hooray! Dependencies dance,
Two revisions waltz in this careful prance,
mdk-core and whitenoise take their leap,
Fresh git commits, promises to keep! ✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The pull request title "Update whitenoise and mdk" directly and accurately reflects the main change in the changeset, which is updating git revision pins for the whitenoise and mdk-core dependencies in rust/Cargo.toml. The title is concise, clear, and specific enough that a teammate scanning history would immediately understand the primary change without ambiguity. It avoids vague terms, emojis, or unnecessary details while capturing the essence of the update.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch bump-mdk-blurhash-fix

📜 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 1510423 and 35f08bf.

⛔ Files ignored due to path filters (1)
  • rust/Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (1)
  • rust/Cargo.toml (2 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: Flutter CI
🔇 Additional comments (2)
rust/Cargo.toml (2)

32-32: The whitenoise revision bump includes the intended blurhash fix with no breaking changes.

The new revision (474fbaec) confirms the blurhash fix in commit "Fix blurhash error bumping mdk and use mls_group_id from Commit to update group image (#389)" along with an associated MDK version bump. Changes are minimal and focused: 3 files modified with 8 net deletions, indicating a clean, targeted fix rather than disruptive refactoring.


19-19: Blurhash fix confirmed in mdk-core revision bump.

The new revision (f46875ec...) includes commit f46875e titled "Fix blurhash generation (#11)", which directly addresses the PR purpose. The CHANGELOG.md has been updated with documentation of the changes. The diff shows 92 insertions and 25 deletions across 7 files, indicating a focused bug fix with no obvious breaking changes.


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

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

@erskingardner erskingardner merged commit 61d76d4 into master Oct 17, 2025
2 checks passed
@erskingardner erskingardner deleted the bump-mdk-blurhash-fix branch October 17, 2025 13:08
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.

3 participants