search

census-instrumentation / opencensus-csharp

Distributed tracing and stats collecting framework
https://opencensus.io
Apache License 2.0
139 stars 32 forks source link

IList sometimes is used in API surface where IEnumerable will suffice #64

Closed SergeyKanzhelev closed 5 years ago

SergeyKanzhelev commented 5 years ago

Reviewing API surface - IList is used when IEnumerable will suffice. For instance in Exporter handler and other places. IList allows manipulating elements - add, remove, etc. when this is absolutely not the intent.

SergeyKanzhelev commented 5 years ago

This will be handled in OpenTelemetry project