Skip to content

Conversation

@sacOO7
Copy link
Collaborator

@sacOO7 sacOO7 commented Sep 25, 2025

Added support for delta encoding using vcdiff

Summary by CodeRabbit

  • Documentation

    • Added a top-level changelog entry for version 1.3.0, updated the Full Changelog range to v1.2.22...v1.3.0, introduced a “Closed issues” note for vcdiff delta support, and retained the existing 1.2.22 section.
  • Chores

    • Bumped the client library version to 1.3.0, updating SDK identification metadata.

@coderabbitai
Copy link
Contributor

coderabbitai bot commented Sep 25, 2025

Walkthrough

Added a new top-level 1.3.0 changelog entry noting vcdiff delta support and updated adjacent Full Changelog link; retained the existing 1.2.22 section. Bumped the Go SDK clientLibraryVersion constant from "1.2.22" to "1.3.0".

Changes

Cohort / File(s) Summary of Changes
Documentation (Changelog)
CHANGELOG.md
Added 1.3.0 entry with a "Closed issues" note (vcdiff delta support #689) and updated the Full Changelog link range to v1.2.22...v1.3.0; kept the existing 1.2.22 section below.
Version bump (Go SDK)
ably/proto_http.go
Updated clientLibraryVersion constant from "1.2.22" to "1.3.0".

Estimated code review effort

🎯 1 (Trivial) | ⏱️ ~2 minutes

Poem

I nibbled the changelog, bright and new,
A hop to one-dot-three with a crunchy view.
Deltas added, issues closed with care,
I twitch my nose and polish my share—
Carrots, versions, all in a row, hooray! 🥕

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
Title Check ✅ Passed The title “Release 1.3.0” concisely and clearly indicates the primary change—the bump to version 1.3.0 and associated release updates—so it accurately reflects the main purpose of the pull request.
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 release/1.2.23

📜 Recent review details

Configuration used: CodeRabbit UI

Review profile: CHILL

Plan: Pro

Disabled knowledge base sources:

  • Jira integration is disabled by default for public repositories

You can enable these sources in your CodeRabbit configuration.

📥 Commits

Reviewing files that changed from the base of the PR and between b3527e2 and 72f0832.

📒 Files selected for processing (2)
  • CHANGELOG.md (1 hunks)
  • ably/proto_http.go (1 hunks)
🚧 Files skipped from review as they are similar to previous changes (2)
  • ably/proto_http.go
  • CHANGELOG.md
⏰ 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). (10)
  • GitHub Check: integration-test (1.19, json)
  • GitHub Check: integration-test (1.19, msgpack)
  • GitHub Check: integration-test (1.20, msgpack)
  • GitHub Check: integration-test (1.24, json)
  • GitHub Check: integration-test (1.25, msgpack)
  • GitHub Check: integration-test (1.23, json)
  • GitHub Check: integration-test (1.23, msgpack)
  • GitHub Check: integration-test (1.24, msgpack)
  • GitHub Check: integration-test (1.20, json)
  • GitHub Check: integration-test (1.25, json)

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.

@sacOO7 sacOO7 requested a review from ttypic September 25, 2025 08:48
@github-actions github-actions bot temporarily deployed to staging/pull/692/features September 25, 2025 08:48 Inactive
@github-actions github-actions bot temporarily deployed to staging/pull/692/godoc September 25, 2025 08:48 Inactive
Copy link
Contributor

@ttypic ttypic left a comment

Choose a reason for hiding this comment

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

LGTM

@sacOO7 sacOO7 merged commit b5c8e55 into main Sep 26, 2025
33 checks passed
@sacOO7 sacOO7 deleted the release/1.2.23 branch September 26, 2025 09:00
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Development

Successfully merging this pull request may close these issues.

3 participants