Skip to content

fix: adjust deno_cache version#31553

Merged
bartlomieju merged 1 commit intodenoland:mainfrom
bartlomieju:fix_dno_cache
Dec 10, 2025
Merged

fix: adjust deno_cache version#31553
bartlomieju merged 1 commit intodenoland:mainfrom
bartlomieju:fix_dno_cache

Conversation

@bartlomieju
Copy link
Copy Markdown
Member

@bartlomieju bartlomieju commented Dec 10, 2025

Small mishap because of #31183 and #31205.

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Dec 10, 2025 •
edited
Loading

Walkthrough

The deno_cache dependency version is bumped from 0.154.0 to 0.155.0 across the workspace. This involves two updates: the workspace dependency specification in the root Cargo.toml and the package version declaration in ext/cache/Cargo.toml. Both files reflect the same version increment while maintaining existing path references and configuration.

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title directly describes the main change: updating the deno_cache version from 0.154.0 to 0.155.0 across two Cargo.toml files.
Description check ✅ Passed The description relates to the changeset by explaining the context (fixes a mishap from earlier PRs) and references the specific PRs that prompted this version adjustment.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
✨ Finishing touches
🧪 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 d5b1548 and cabeaea.

⛔ Files ignored due to path filters (1)
  • Cargo.lock is excluded by !**/*.lock
📒 Files selected for processing (2)
  • Cargo.toml (1 hunks)
  • ext/cache/Cargo.toml (1 hunks)
🧰 Additional context used
🧠 Learnings (1)
📓 Common learnings 
Learnt from: CR
Repo: denoland/deno PR: 0
File: CLAUDE.md:0-0
Timestamp: 2025-11-24T16:19:37.808Z
Learning: Use `cargo update` to update Cargo dependencies to latest compatible versions
⏰ 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). (11)
  • GitHub Check: test release linux-x86_64
  • GitHub Check: test debug linux-aarch64
  • GitHub Check: test debug linux-x86_64
  • GitHub Check: test release windows-x86_64
  • GitHub Check: test debug windows-x86_64
  • GitHub Check: test debug macos-x86_64
  • GitHub Check: test debug macos-aarch64
  • GitHub Check: lint debug macos-x86_64
  • GitHub Check: lint debug windows-x86_64
  • GitHub Check: lint debug linux-x86_64
  • GitHub Check: build libs
🔇 Additional comments (2)
ext/cache/Cargo.toml (1)

5-5: Version bump is consistent with root workspace dependency.

Package version aligns with the workspace dependency specification in root Cargo.toml.

Cargo.toml (1)

92-92: Workspace dependency version is bumped consistently with ext/cache package.

The version and path remain correctly specified. The change updates deno_cache from 0.154.0 to 0.155.0 as an isolated modification to the workspace dependency declaration.

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.

@bartlomieju bartlomieju enabled auto-merge (squash) December 10, 2025 14:47
@bartlomieju bartlomieju merged commit d94c7fa into denoland:main Dec 10, 2025
31 of 37 checks passed
@bartlomieju bartlomieju deleted the fix_dno_cache branch December 10, 2025 16:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@dsherret dsherret dsherret approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@bartlomieju @dsherret