Skip to content

Conversation

ChiragAgg5k
Copy link
Member

No description provided.

Copy link
Contributor

coderabbitai bot commented Sep 18, 2025

📝 Walkthrough

Walkthrough

The change updates a configuration value in app/config/platforms.php. Specifically, in the APP_PLATFORM_CONSOLE configuration under sdks -> cli, the version string is changed from '9.1.0' to '10.0.0'. No other files or logic are modified.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Possibly related PRs

Suggested reviewers

  • christyjacob4
  • Meldiron

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description Check ❓ Inconclusive No pull request description was provided, so there is insufficient information about the motivation, testing, or compatibility impact of the version bump even though the diff shows a simple data/config change. Please add a brief description explaining why the CLI was bumped, any compatibility or migration considerations, and what testing was performed so reviewers can assess risk and release-note impact.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "update cli to 10.0.0" is concise and directly reflects the main change in the changeset (the CLI SDK version bump in app/config/platforms.php from 9.1.0 to 10.0.0), making it clear to reviewers what the PR does.
Docstring Coverage ✅ Passed No functions found in the changes. Docstring coverage check skipped.
✨ Finishing touches
  • 📝 Generate Docstrings
🧪 Generate unit tests
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch update-cli-10.0.0

📜 Recent review details

Configuration used: Path: .coderabbit.yaml

Review profile: CHILL

Plan: Pro

📥 Commits

Reviewing files that changed from the base of the PR and between 34f3501 and 427c891.

⛔ Files ignored due to path filters (2)
  • composer.lock is excluded by !**/*.lock
  • docs/sdks/cli/CHANGELOG.md is excluded by !docs/sdks/**
📒 Files selected for processing (1)
  • app/config/platforms.php (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). (3)
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: Setup & Build Appwrite Image
  • GitHub Check: scan

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.

Copy link

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
golang.org/x/crypto v0.31.0 CVE-2025-22869 HIGH
golang.org/x/oauth2 v0.24.0 CVE-2025-22868 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

Copy link

✨ Benchmark results

  • Requests per second: 1,206
  • Requests with 200 status code: 217,111
  • P99 latency: 0.161625638

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,206 986
200 217,111 177,451
P99 0.161625638 0.196820867

@loks0n loks0n merged commit 6ff6839 into 1.8.x Sep 22, 2025
41 checks passed
@stnguyen90 stnguyen90 deleted the update-cli-10.0.0 branch October 1, 2025 23:58
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.

2 participants