Skip to content

SDK releases#10539

Merged
abnegate merged 1 commit into1.8.xfrom
chore-sdk-release
Sep 23, 2025
Merged

SDK releases#10539
abnegate merged 1 commit into1.8.xfrom
chore-sdk-release

Conversation

@abnegate
Copy link
Copy Markdown
Member

@abnegate abnegate commented Sep 23, 2025

What does this PR do?

(Provide a description of what this PR does and why it's needed.)

Test Plan

(Write your test plan here. If you changed any code, please provide us with clear instructions on how you verified your changes work. Screenshots may also be helpful.)

Related PRs and Issues

  • (Related PR or issue)

Checklist

  • Have you read the Contributing Guidelines on issues?
  • If the PR includes a change to an API's metadata (desc, label, params, etc.), does it also include updated API specs and example docs?

@coderabbitai
Copy link
Copy Markdown
Contributor

coderabbitai bot commented Sep 23, 2025 •
edited
Loading

📝 Walkthrough

Walkthrough

This PR updates SDK version strings in app/config/platforms.php. For each listed platform, the public SDK version is incremented by a minor version (e.g., 20.0.0 -> 20.1.0). No other fields (such as urls, packages, enabled flags) are modified. There are no structural changes, no logic or control-flow edits, and no alterations to public interfaces.

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~3 minutes

Possibly related PRs

Suggested reviewers

  • loks0n
  • ChiragAgg5k
  • christyjacob4

Pre-merge checks and finishing touches

❌ Failed checks (1 inconclusive)
Check name Status Explanation Resolution
Description Check ❓ Inconclusive The PR description contains only the repository contribution template with empty sections and does not describe the applied SDK version bumps, rationale, or test/verification steps, so it is too vague to assess the changes from the description alone. Please update the PR description to summarize the changes (which SDK versions were bumped and why), include a short test plan or verification steps, link any related issues/PRs, and complete the checklist items before merging.
✅ Passed checks (2 passed)
Check name Status Explanation
Title Check ✅ Passed The title "SDK releases" concisely and accurately summarizes the main change — bumping multiple platform SDK version numbers in app/config/platforms.php; it is short, focused, and clearly related to the changeset.
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 chore-sdk-release

Warning

Review ran into problems

🔥 Problems

Git: Failed to clone repository. Please run the @coderabbitai full review command to re-trigger a full review. If the issue persists, set path_filters to include or exclude specific files.

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.

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 23, 2025

Security Scan Results for PR

Docker Image Scan Results

Package Version Vulnerability Severity
libexpat 2.7.1-r0 CVE-2025-59375 HIGH
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-47906 HIGH
stdlib 1.22.10 CVE-2025-47907 HIGH

Source Code Scan Results

🎉 No vulnerabilities found!

coderabbitai[bot]
coderabbitai bot reviewed Sep 23, 2025
View reviewed changes
Copy link
Copy Markdown
Contributor

@coderabbitai coderabbitai bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Actionable comments posted: 0

🧹 Nitpick comments (1)
app/config/platforms.php (1)

14-14: Optional: automate version sync to avoid drift.

Consider deriving these values from registry APIs or a single manifest in release automation to prevent manual mismatches.

Also applies to: 265-265

📜 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 dce0149 and 0995bd6.

⛔ Files ignored due to path filters (11)
  • composer.lock is excluded by !**/*.lock
  • docs/examples/1.8.x/server-graphql/examples/functions/create-duplicate-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/functions/create-template-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/functions/create-vcs-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/functions/update-deployment-status.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/sites/create-duplicate-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/sites/create-template-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/sites/create-vcs-deployment.md is excluded by !docs/examples/**
  • docs/examples/1.8.x/server-graphql/examples/sites/update-deployment-status.md is excluded by !docs/examples/**
  • docs/sdks/dart/CHANGELOG.md is excluded by !docs/sdks/**
  • docs/sdks/flutter/CHANGELOG.md is excluded by !docs/sdks/**
📒 Files selected for processing (1)
  • app/config/platforms.php (14 hunks)
🔇 Additional comments (3)
app/config/platforms.php (3)

14-14: Client SDK version bumps: LGTM.

Confirm corresponding releases are live (npm/pub/maven/GitHub) and changelogs updated.

Also applies to: 63-63, 82-82, 119-119, 142-142

265-265: Server SDK version bumps: LGTM.

Please verify Node/PHP/Python/Ruby/Go/.NET/Dart/Kotlin/Swift releases and tags exist; sync docs/CHANGELOGs.

Also applies to: 284-284, 303-303, 322-322, 341-341, 360-360, 379-379, 398-398, 421-421

14-14: Verify all bumped SDK versions exist in their registries/tags (script included).

Looks good. Please confirm each version is published and tagged to avoid broken docs/install commands.

Run this script to validate against GitHub tags and major package registries:

Also applies to: 63-63, 82-82, 119-119, 142-142, 265-265, 284-284, 303-303, 322-322, 341-341, 360-360, 379-379, 398-398, 421-421

@github-actions
Copy link
Copy Markdown

github-actions bot commented Sep 23, 2025

✨ Benchmark results

  • Requests per second: 1,193
  • Requests with 200 status code: 214,794
  • P99 latency: 0.161868897

⚡ Benchmark Comparison

Metric This PR Latest version
RPS 1,193 968
200 214,794 174,354
P99 0.161868897 0.194260854

@abnegate abnegate merged commit 5448c79 into 1.8.x Sep 23, 2025
41 checks passed
@abnegate abnegate deleted the chore-sdk-release branch September 23, 2025 07:41
This was referenced Sep 26, 2025
Merged
Merged
This was referenced Oct 13, 2025
Merged
Merged
@coderabbitai coderabbitai bot mentioned this pull request Nov 4, 2025
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@coderabbitai coderabbitai[bot] coderabbitai[bot] left review comments

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.

1 participant

@abnegate