Skip to content

feat: Implement SDK spec v0.7.0 - #248

Closed
flagsmithsmith[bot] wants to merge 4 commits into
mainfrom
openhands/sdk-spec-v0.7.0
Closed

feat: Implement SDK spec v0.7.0#248
flagsmithsmith[bot] wants to merge 4 commits into
mainfrom
openhands/sdk-spec-v0.7.0

Conversation

@flagsmithsmith

@flagsmithsmith flagsmithsmith Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor

Implements the SDK spec's v0.7.0 in this SDK.

This pull request was opened by an AI agent (OpenHands).

renovate Bot and others added 4 commits October 29, 2026 19:54
@renovate
deps: update dependency flagsmith/sdk-spec to v0.7.0
28bfbdb
@flagsmithsmith @claude
test(sdk-spec): Run the experimentation steps against the SDK
0c48af8 
Spec v0.6.0 adds scenarios for getting an experiment flag, tracking an
event and tracking an exposure event, all with events enabled. The SDK
already implements these (#226), so the steps file wires each step to
FlagsmithClient.getExperimentFlag, trackEvent and trackExposureEvent, and
creates the events-enabled client through FlagsmithConfig.

Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
(cherry picked from commit f5f3801)
@khvn26
Merge branch 'main' into renovate/sdk-spec
25d854f
@khvn26
ci: run events SDK spec tests
48c35f4
@flagsmithsmith flagsmithsmith Bot added the sdk-spec Bumps of the Flagsmith SDK behavioural spec label Nov 5, 2026
@flagsmithsmith flagsmithsmith Bot mentioned this pull request Nov 5, 2026
Closed
1 task
@coderabbitai

coderabbitai Bot commented Nov 5, 2026

Copy link
Copy Markdown

Important

Review skipped

Bot user detected.

To trigger a single review, invoke the @coderabbitai review command.

Run configuration

  • Source mode: Auto
  • Repository: Unavailable
  • Branch: Unavailable
  • File: No configuration file from this repository applied
  • Commit: Unavailable
  • Layers: Organization UI
  • Review profile: ASSERTIVE
  • Plan: Advanced
  • Run ID: 2edecb17-de32-4bb1-a4c5-324fbf8e0332

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • Trigger review
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@flagsmithsmith

flagsmithsmith Bot commented Nov 5, 2026

Copy link
Copy Markdown
Contributor Author

Superseded by #250, which carries this pull request's implementation work, for the SDK spec's latest version.

@flagsmithsmith flagsmithsmith Bot closed this Nov 5, 2026
@flagsmithsmith
flagsmithsmith Bot deleted the openhands/sdk-spec-v0.7.0 branch November 5, 2026 17:57
@codspeed

codspeed Bot commented Nov 5, 2026

Copy link
Copy Markdown

Merging this PR will not alter performance

✅ 15 untouched benchmarks

Comparing openhands/sdk-spec-v0.7.0 (48c35f4) with main (82ea4c5)1

Open in CodSpeed

Footnotes

  1. No successful run was found on main (e278fd5) during the generation of this report, so 82ea4c5 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

sdk-spec Bumps of the Flagsmith SDK behavioural spec

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@khvn26