Workiva / opentelemetry-dart

Apache License 2.0
58 stars 33 forks source link

O11Y-4010: Add ContextManager for Context #167

Closed changliu-wk closed 3 months ago

changliu-wk commented 4 months ago

Which problem is this PR solving?

Short description of the change

How Has This Been Tested?

CI test would be efficient.

Checklist:

aviary2-wf commented 4 months ago

Security Insights

No security relevant content was detected by automated scans.

Action Items

changliu-wk commented 4 months ago

After discussed with @blakeroberts-wk, we decided to revert the breaking changes, we will have follow up tickets about the API redesign.

jonathancampbell-wk commented 4 months ago

After discussed with @blakeroberts-wk, we decided to revert the breaking changes, we will have follow up tickets about the API redesign.

Which changes do we want to push to future PRs?

changliu-wk commented 4 months ago

Which changes do we want to push to future PRs?

I think we would like to mimic what opentelemetry-js does, with some adjustments, like not having delete method within context api.

changliu-wk commented 3 months ago

@Workiva/release-management-p

changliu-wk commented 3 months ago

QA+1