-
Notifications
You must be signed in to change notification settings - Fork 4.9k
chore: rc releases for cli sdk #11016
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
base: 1.8.x
Are you sure you want to change the base?
Conversation
📝 WalkthroughWalkthroughThe pull request updates the Console/Command Line SDK entry in the configuration file by changing the version number from 12.0.1 to 13.0.0-rc.2. This is a single-line modification to a version string within a SDK registry. No configuration URLs, enable flags, metadata, or other entries are affected by this change. Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Pre-merge checks and finishing touches❌ Failed checks (1 inconclusive)
✅ Passed checks (2 passed)
✨ Finishing touches
🧪 Generate unit tests (beta)
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. Comment |
Security Scan Results for PRDocker Image Scan Results
Source Code Scan Results🎉 No vulnerabilities found! |
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: Path: .coderabbit.yaml
Review profile: CHILL
Plan: Pro
⛔ Files ignored due to path filters (2)
composer.lockis excluded by!**/*.lockdocs/sdks/cli/CHANGELOG.mdis excluded by!docs/sdks/**
📒 Files selected for processing (1)
app/config/sdks.php
⏰ 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). (3)
- GitHub Check: Setup & Build Appwrite Image
- GitHub Check: Setup & Build Appwrite Image
- GitHub Check: scan
| 'key' => 'cli', | ||
| 'name' => 'Command Line', | ||
| 'version' => '12.0.1', | ||
| 'version' => '13.0.0-rc.2', |
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.
🧩 Analysis chain
🏁 Script executed:
# First, verify the file and line content
cat -n app/config/sdks.php | sed -n '225,235p'Repository: appwrite/appwrite
Length of output: 561
🏁 Script executed:
# Check npm registry for version 13.0.0-rc.2 availability
curl -s https://registry.npmjs.org/appwrite-cli/13.0.0-rc.2 | head -20Repository: appwrite/appwrite
Length of output: 93
🏁 Script executed:
# Search for hardcoded version references in codebase
rg "12\.0\.1|13\.0\.0" --type php --type js --type ts -nRepository: appwrite/appwrite
Length of output: 128
Verify npm version exists before merging—version 13.0.0-rc.2 not found in npm registry.
The version 13.0.0-rc.2 referenced on line 229 does not exist in the npm registry. Either publish this version to npm or update the config to an existing version before this change can be merged.
🤖 Prompt for AI Agents
In app/config/sdks.php around line 229, the SDK version is set to '13.0.0-rc.2'
which does not exist in the npm registry; update this entry to a valid,
published npm version (e.g., an existing stable or RC version) or coordinate to
publish '13.0.0-rc.2' before merging. Replace the value with the confirmed
version string and run composer/npm install or lockfile update as appropriate to
ensure dependency resolution succeeds.
✨ Benchmark results
⚡ Benchmark Comparison
|
No description provided.