You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I would propose we rename the package to view and just export a Config type. That way func WithViews(views ...view.Config) Option in the sdk/metric reads without a stutter. Additionally, if we then have New(...Option) Config in the view package the call would be WithViews(view.New(...), view.New(...) and this would match our existing configuration patterns.
I'm wondering if all this view configuration structure is needed. The view is ultimately only going to be passed to a MeterProvider as an option. Why not define it as a type directly in the sdk/metric package with exported fields for all configuration options and an associated NewView function that returns a View with all the defaults set?
Blocked by #2799sdk/metric/view
package.TODO
s have an issue tracking them.The text was updated successfully, but these errors were encountered: