Skip to content

Conversation

@cwaldren-ld
Copy link
Contributor

@cwaldren-ld cwaldren-ld commented Dec 18, 2023

This PR updates the Lua SDK to build against the C++ Server-side SDK with Redis v3.x via its provided C bindings.

It will be the first v2 release of this SDK.

The shape of the SDK mostly remains the same, with the major changes being:

  • Users are replaced with contexts (makeUser is a wrapper, addedmakeContext)
  • Configuration overhaul (nested configurations, type checking, data system)
  • Removed obsolete config options and methods (like alias)
  • Removed global logger config

@cwaldren-ld cwaldren-ld changed the title feat!: use C++ Server-side SDK 3.0 bindings feat!: use C++ Server-side SDK 3.0 bindings [v2] Dec 27, 2023
@cwaldren-ld cwaldren-ld marked this pull request as ready for review December 27, 2023 20:54
@cwaldren-ld cwaldren-ld requested a review from a team December 27, 2023 20:54
@cwaldren-ld cwaldren-ld merged commit 95e9718 into main Dec 28, 2023
@cwaldren-ld cwaldren-ld deleted the cw/sc-227439/updated-c-bindings branch December 28, 2023 19:58
cwaldren-ld pushed a commit that referenced this pull request Dec 28, 2023
🤖 I have created a release *beep* *boop*
---


##
[2.0.0](1.2.2...v2.0.0)
(2023-12-28)


### ⚠ BREAKING CHANGES

* SDK configuration exposes new options and is organized hierarchically
* remove `inlineUsersInEvents` and `userKeysFlushInterval` config
options
* remove global `registerLogger` function, replace with config option
* remove `alias` function, replace usage with multi-kind contexts
* Variation and VariationDetail now take contexts
* `makeUser` behavior modified to construct a user-kind context
* use C++ Server-side SDK 3.0 bindings
([#31](#31))

### Features

* added `makeContext` for constructing single or multi-kind contexts
([95e9718](95e9718))
* use C++ Server-side SDK 3.0 bindings
([#31](#31))
([95e9718](95e9718))


### Code Refactoring

* makeUser behavior modified to construct a user-kind context
([95e9718](95e9718))
* remove `alias` function, replace usage with multi-kind contexts
([95e9718](95e9718))
* remove `inlineUsersInEvents` and `userKeysFlushInterval` config
options
([95e9718](95e9718))
* remove global `registerLogger` function, replace with config option
([95e9718](95e9718))
* SDK configuration exposes new options and is organized hierarchically
([95e9718](95e9718))
* Variation and VariationDetail now take contexts
([95e9718](95e9718))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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.

9 participants