Skip to content

feat: eval reasons #179

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 12 commits into from
Jul 28, 2025
Merged

feat: eval reasons #179

merged 12 commits into from
Jul 28, 2025

Conversation

@kaushalkapasi
Copy link
Contributor

@kaushalkapasi kaushalkapasi commented Jul 24, 2025 •
edited
Loading

  • chore: update WASM Bucketing lib to 1.41.0
  • feat: adds Eval Reasons support to Local Bucketing Client
  • chore: update Local Bucketing tests for validating Eval Reasons
  • feat: adds Eval Reasons support to Cloud Bucketing Client
  • chore: adds temporary override to test-harness capabilities to verify eval reason
  • feat: adds Eval Reasons to allVariables method
  • fix: use ErrorMessage enum to set error details generated by the SDK
  • chore: update variable protobuf object to return _feature and include corresponding varableFeatureIds test harness capability

@kaushalkapasi kaushalkapasi requested a review from a team as a code owner July 24, 2025 20:44
@kaushalkapasi kaushalkapasi requested review from a team and JamieSinn July 24, 2025 20:45
JamieSinn
JamieSinn approved these changes Jul 24, 2025
View reviewed changes
phaym
phaym reviewed Jul 24, 2025
View reviewed changes
@phaym
Copy link

phaym commented Jul 25, 2025 •
edited
Loading

open feature? nvm we discussed, it will be in another pr

@kaushalkapasi kaushalkapasi force-pushed the feat-eval-reasons branch 2 times, most recently from 7b78625 to fce30ae Compare July 25, 2025 19:35
@kaushalkapasi kaushalkapasi requested review from JamieSinn and phaym July 25, 2025 20:50
jsalaber
jsalaber reviewed Jul 25, 2025
View reviewed changes
@kaushalkapasi kaushalkapasi requested a review from jsalaber July 25, 2025 21:04
@kaushalkapasi kaushalkapasi merged commit 86a1614 into main Jul 28, 2025
7 of 8 checks passed
@kaushalkapasi kaushalkapasi deleted the feat-eval-reasons branch July 28, 2025 13:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@phaym phaym phaym approved these changes

@jsalaber jsalaber jsalaber approved these changes

@JamieSinn JamieSinn Awaiting requested review from JamieSinn

Assignees

No one assigned

Labels

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

@kaushalkapasi @phaym @JamieSinn @jsalaber