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

refactor: Add Confidence Library scaffolding #83

Merged
merged 9 commits into from
Apr 2, 2024
Merged

Conversation

fabriziodemaria
Copy link
Member

@fabriziodemaria fabriziodemaria commented Mar 27, 2024

Introducing a new Confidence Library. The Confidence Library conforms to a ConfidenceEventSender protocol, allowing the hosting app to emit events to Confidence (implementation of the actual event sending to the backend will be added in following PRs).

The change is almost backwards compatible: the user has to add the Confidence library as a dependency and import, but no change is required in the main code. However, it's totally ok to configure the Provider with the Confidence object, with available configurations:

  • Secret
  • Timeout
  • Region
  • InitializationStrategy

For any other testing-oriented dependency injection, users can still use the current Builder pattern of the Provider.

Note that this PR also includes a very incomplete setup for Contextual and for adding the EvaluationContext to each event. This will be further implemented in following PRs.

@fabriziodemaria fabriziodemaria changed the title [WIP] Add Confidence Library feat: [WIP] Add Confidence Library Mar 27, 2024
@fabriziodemaria fabriziodemaria force-pushed the confidence-lib branch 3 times, most recently from 7a67af9 to 2f2abe1 Compare March 28, 2024 10:16
@fabriziodemaria fabriziodemaria force-pushed the confidence-lib branch 6 times, most recently from bef7f32 to 3f0eba0 Compare March 28, 2024 14:52
@fabriziodemaria fabriziodemaria changed the title feat: [WIP] Add Confidence Library refactor: Add Confidence Library scaffolding Mar 28, 2024
@fabriziodemaria fabriziodemaria marked this pull request as ready for review March 28, 2024 15:34
nicklasl
nicklasl previously approved these changes Apr 2, 2024
@fabriziodemaria fabriziodemaria merged commit 2e49e23 into main Apr 2, 2024
3 checks passed
@fabriziodemaria fabriziodemaria deleted the confidence-lib branch April 2, 2024 13:20
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.

2 participants