feat: Add OpenFeature.Extensions.Hosting package
Closes: open-feature/dotnet-sdk#264
Related
- open-feature/spec#81
- open-feature/ofep#32
- open-feature/spec#227
TODO:
- [ ] moar tests
- [ ] evaluation context
- [ ]
IHostingEnvironmentsample
- [ ]
- [ ] hooks
- [ ] providers:
- [ ] none
- [ ] one
- [ ] multiple
- [ ] evaluation context
- [ ] xml docs
- [ ] readme docs
Codecov Report
Attention: Patch coverage is 65.78947% with 39 lines in your changes missing coverage. Please review.
Project coverage is 90.82%. Comparing base (
11a0333) to head (bcf19c4). Report is 11 commits behind head on main.
:exclamation: Current head bcf19c4 differs from pull request most recent head 9daa33f
Please upload reports for the commit 9daa33f to get more accurate results.
Additional details and impacted files
@@ Coverage Diff @@
## main #181 +/- ##
==========================================
- Coverage 95.40% 90.82% -4.59%
==========================================
Files 27 29 +2
Lines 1111 1068 -43
Branches 120 113 -7
==========================================
- Hits 1060 970 -90
- Misses 34 70 +36
- Partials 17 28 +11
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
FYI couple of known test failures:
- [x] ~~DCO is intentionally broken by a temp commit containing a squash of #187 and #188. Once those PRs are merged, I'll rebase and elide the temp commit.~~
- [x] dotnet-format is being weird about
OpenFeatureHostedServiceand I'm pretty sure its just a sign that we need to migrate from thedotnet-formattool, to the newdotnet formatthat's baked into recent SDKs. - [x] ~~flaky unit tests: pretty sure these are all related to the async stuff, so hoping they'll be less common after open-feature/dotnet-sdk-contrib#184 lands, but in the meantime please just re-run the test if its failed at the time of your review.~~
Is this PR going to be merged? I was looking forward to this capabilities with version 2.0
Is this PR going to be merged? I was looking forward to this capabilities with version 2.0
@nbr28 This is still being worked on, mostly by @askpt :pray: . We are still planning to merge once it's completed.
Superseded by #310