-
Notifications
You must be signed in to change notification settings - Fork 17
Bump ardrive-core-js and cli version to 3.0.5 #372
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
Conversation
WalkthroughThe PR bumps the package version from 3.0.4 to 3.0.5 and updates the ardrive-core-js dependency to the matching 3.0.5 version in package.json. Changes
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested reviewers
Poem
Pre-merge checks and finishing touches✅ Passed checks (3 passed)
✨ Finishing touches🧪 Generate unit tests (beta)
Comment |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Actionable comments posted: 1
📜 Review details
Configuration used: CodeRabbit UI
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
.yarn/cache/ardrive-core-js-npm-3.0.5-29d8269aeb-c0dbf53ddb.zipis excluded by!**/.yarn/**,!**/*.zipyarn.lockis excluded by!**/yarn.lock,!**/*.lock
📒 Files selected for processing (1)
package.json(1 hunks)
| "version": "3.0.5", | ||
| "description": "The ArDrive Command Line Interface (CLI is a Node.js application for terminal-based ArDrive workflows. It also offers utility operations for securely interacting with Arweave wallets and inspecting various Arweave blockchain conditions.", | ||
| "main": "./lib/index.js", | ||
| "bin": { | ||
| "ardrive": "./lib/index.js" | ||
| }, | ||
| "types": "./lib/index.d.ts", | ||
| "dependencies": { | ||
| "ardrive-core-js": "3.0.4", | ||
| "ardrive-core-js": "3.0.5", | ||
| "arweave": "1.15.7", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Update the lockfile to pick up 3.0.5.
Bumping ardrive-core-js here is not enough—without the corresponding yarn.lock/package-lock.json update, installs will still resolve to 3.0.4, so the release won’t actually ship the new core. Please regenerate the lockfile (e.g., yarn install) and include it in the PR.
🤖 Prompt for AI Agents
In package.json around lines 3 to 12, you bumped ardrive-core-js to 3.0.5 but
did not update the lockfile, so installs will still pull 3.0.4; regenerate and
commit the appropriate lockfile (run yarn install if using Yarn or npm install
if using npm) so yarn.lock or package-lock.json reflects 3.0.5, verify the
lockfile contains ardrive-core-js@3.0.5, and include that updated lockfile in
the PR before merging.
Summary by CodeRabbit