André Silva

Results 37 comments of André Silva

1 - @open-feature/sdk-dotnet-approvers & @open-feature/sdk-dotnet-maintainers: if this makes sense, I'll abandon https://github.com/open-feature/dotnet-sdk-contrib/pull/122 and move the OTEL stuff here. Anyone against? 2 - What are the steps to deprecate the library...

> > I'm personally finding it challenging to answer questions about perf, overhead, etc., related to the impact of using the OpenFeature SDK on a range of luke-warn to scorchingly-hot...

@toddbaert and @beeme1mr the new spec changes (https://github.com/open-feature/spec/pull/287) will help close this issue?

> I think we can propose the creation of updated hooks directly in the SDK, since there's no additional deps required, which I guess would close this. > > What...

I wonder if we should kill `Hosting` and move this logic to the `DependencyInjection` library. Also, I am thinking of doing this ```cs builder.Services.AddOpenFeature(featureBuilder => { featureBuilder .AddHostedFeatureLifecycle() // Remove...

@arttonoyan and @kylejuliandev, I was playing around with Copilot to generate some documentation for issue #471. I think it was a great suggestion in relation to the `AddHostedLifecycle` issue we...

> @askpt and @kylejuliandev I've been reflecting on this topic and exploring how other frameworks handle similar scenarios. Previously, I advocated for maintaining two separate packages: `OpenFeature.DependencyInjection` and `OpenFeature.Hosting`, to...

@WeihanLi, @kylejuliandev, @arttonoyan Let's follow this recommendation: https://github.com/open-feature/dotnet-sdk/issues/472#issuecomment-2937082094 We should keep the DI package and deprecate the Hosting one. Let's also make sure the `AddHosting` is no longer required. The...

Hey, @kylejuliandev! Great work on this PR; everything seems good to go. As you pointed out, however, a few unit tests are missing to get this PR out. Would you...

@toddbaert, @beeme1mr, @kinyoklion, @kylejuliandev, @arttonoyan and @austindrenski I merged the OTEL semcov from #397 and replaced the similar attributes. Please have a look and let me know what you think.