[flagsmithsmith]

Contributes to Flagsmith/sdk-spec#13

[coderabbitai]

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: 24d83659-5fd6-4391-9836-d69c3b6bfd4d

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

• X
• Mastodon
• Reddit
• LinkedIn

[codspeed]

Merging this PR will not alter performance

1 warning

⚠️ 1 benchmark is dominated by system calls

System calls cannot be simulated consistently and are left out of the measure. The reported value understates the real cost. Switch to the Walltime instrument to measure them.

Learn more about system calls

See affected benchmarks

✅ 15 untouched benchmarks

---

_{Comparing openhands/sdk-spec-v0.7.1 (b22894f) with main (af82ba5)}

[flagsmithsmith]

This pull request has nothing left to do: SDK spec v0.7.1 is already implemented on main and was released as 10.0.0.

• The work already landed. feat!: Support getExperimentFlag, trackEvent and trackExposure methods #214 (feat!: Support getExperimentFlag, trackEvent and trackExposure methods) was merged at 05:00:34Z and released in chore(main): release 10.0.0 #215 (chore(main): release 10.0.0). main already runs run-spec.yml@v0.7.1 with tags: core or events or local-evaluation.
• Merging this changes nothing. git merge-tree --write-tree origin/main origin/openhands/sdk-spec-v0.7.1 gives tree 1d4bd25, the same as main's own tree. The diff shown here is against the old merge base 745945c. Those changes are already on main, through feat!: Support getExperimentFlag, trackEvent and trackExposure methods #214's squash.
• CI is green, not red. Every check passes, including sdk-spec / Spec v0.7.1 (run 42759429802). The pull_request checks run on the merge with main, which already contains the events implementation.
• How it was opened. The branch's activity log shows its deletion when feat!: Support getExperimentFlag, trackEvent and trackExposure methods #214 merged (05:00:36Z, 7086f72 -> deleted). Then flagsmithsmith[bot] recreated it at b22894f (05:00:48Z), the commit feat!: Support getExperimentFlag, trackEvent and trackExposure methods #214 started from, and opened this PR at 05:00:56Z.

So I haven't pushed any changes. Whether to close this PR is up to the maintainers. It may also be worth checking why the automation reopens a spec PR after its branch has merged.