Skip to content
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

build(meta-provider): Support non-breaking versions of openfeature-sdk #47

Merged
merged 2 commits into from
Feb 28, 2025

Conversation

jblacker
Copy link
Contributor

@jblacker jblacker commented Feb 6, 2025

This PR

The meta provider's gemspec stated only v0.3.x could be used. Unfortunately, this conflicts with the official LaunchDarkly provider, which uses version 0.4.0, preventing this gem from being used along with it.

Although technically speaking openfeature-sdk 0.3.0 -> 0.4.0 is a breaking change because of the change of string values, but it should not cause a problem. This is because the provider uses only the constants rather than the string values themselves, so this should not break the provider.

Notes

I put <= 0.4 as the maximum version of the SDK, but this can be changed if ya'll think it should be a different value.

How to test

Import this branch into an application along with the the gem built from https://github.com/launchdarkly/openfeature-ruby-server and verify that using it within a meta-provider works correctly.

The meta provider's gemspec stated only v0.3.x could be used. Unfortunately, this
conflicts with the official LaunchDarkly provider, which uses version 0.4.0, preventing
this gem from being used along with it. This update allows either 0.3.0 or 0.4.0 to be used.

Although technically speaking openfeature-sdk 0.3.0 -> 0.4.0 is a breaking change
because of the change of string values. However, the provider uses only the constants rather than
the string values so this should not break the provider.

Signed-off-by: Jordan Blacker <jblacker@justworks.com>
@jblacker jblacker changed the title deps(meta-provider): Support non-breaking versions of openfeature-sdk build(meta-provider): Support non-breaking versions of openfeature-sdk Feb 6, 2025
@beeme1mr beeme1mr requested a review from maxveldink February 6, 2025 20:13
Signed-off-by: Jordan Blacker <jblacker@justworks.com>
@jblacker jblacker force-pushed the update-required-openfeature-sdk branch from 62b9ecc to db9f770 Compare February 6, 2025 21:38
@nathan-beam
Copy link

This is blocking work for us too, this unofficial Flagsmith provider depends on openfeature-sdk ~> 0.4.0

@thomaspoignant @beeme1mr tagging for visibility

Copy link
Member

@beeme1mr beeme1mr left a comment

Choose a reason for hiding this comment

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

Sorry for the delay! This looks good to me. We'll just need to keep track of future updates to the Ruby SDK. A 1.0 release would make this a bit easier.

@beeme1mr beeme1mr merged commit 5a98cf4 into open-feature:main Feb 28, 2025
11 checks passed
beeme1mr pushed a commit that referenced this pull request Feb 28, 2025
🤖 I have created a release *beep* *boop*
---


##
[0.0.4](openfeature-meta_provider/v0.0.3...openfeature-meta_provider/v0.0.4)
(2025-02-28)


### 🛠️ Build

* **meta-provider:** Support non-breaking versions of openfeature-sdk
([#47](#47))
([5a98cf4](5a98cf4))

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

Signed-off-by: OpenFeature Bot <109696520+openfeaturebot@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.

3 participants