Skip to content

Conversation

@nnnkkk7
Copy link
Contributor

@nnnkkk7 nnnkkk7 commented May 8, 2025

This PR introduces the ability to set the SDK version through options when creating a new SDK instance.
This change is necessary for building an OpenFeature provider to set SDK version .

@nnnkkk7 nnnkkk7 force-pushed the add/versionid-args branch 2 times, most recently from ba77485 to 24855e0 Compare May 9, 2025 00:44
Comment on lines -30 to -34
type registerEventsResponseError struct {
Retriable bool `json:"retriable,omitempty"`
Message string `json:"message,omitempty"`
}

Copy link
Contributor Author

Choose a reason for hiding this comment

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

This struct is unused now.

@nnnkkk7 nnnkkk7 marked this pull request as ready for review May 9, 2025 01:24
@nnnkkk7 nnnkkk7 requested review from cre8ivejp, hvn2k1 and kakcy as code owners May 9, 2025 01:24
@nnnkkk7 nnnkkk7 force-pushed the add/versionid-args branch from 24855e0 to 5e6956f Compare May 9, 2025 01:36
@cre8ivejp
Copy link
Member

Just confirming. Are you going to add the source ID option in another PR, right?

Copy link
Member

@cre8ivejp cre8ivejp left a comment

Choose a reason for hiding this comment

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

Nice work!

@nnnkkk7
Copy link
Contributor Author

nnnkkk7 commented May 9, 2025

@cre8ivejp
I haven't been able to send comments🙏
Yes, that's correct.
I will make another PR for the sourceid, similar to this PR.
I split the PRs to make the review easier.

@nnnkkk7 nnnkkk7 merged commit 2655753 into master May 9, 2025
7 checks passed
@nnnkkk7 nnnkkk7 deleted the add/versionid-args branch May 9, 2025 06:19
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