Skip to content
This repository has been archived by the owner on Sep 6, 2022. It is now read-only.

Remove opencensus dep #58

Closed
Stebalien opened this issue Sep 19, 2019 · 0 comments · Fixed by #208
Closed

Remove opencensus dep #58

Stebalien opened this issue Sep 19, 2019 · 0 comments · Fixed by #208
Labels
kind/enhancement A net-new feature or improvement to an existing feature

Comments

@Stebalien
Copy link
Member

Unfortunately, this means that anything wanting to use the libp2p interfaces (even light-clients that leave all the heavy lifting up to a daemon) need to import opencensus.

From the looks of it, we aren't actually using any of the code in metrics/register.go. We might want to consider removing this code unless we think we're actually going to use it.

Note: this pulls in stuff like grpc, google's cloud API libraries, etc.

@Stebalien Stebalien added the kind/enhancement A net-new feature or improvement to an existing feature label Sep 19, 2019
Stebalien added a commit that referenced this issue Jul 22, 2021
This drops the opensensus dependency. Nobody uses this, from what I can
tell.

fixes #58
Stebalien added a commit that referenced this issue Jul 23, 2021
This drops the opensensus dependency. Nobody uses this, from what I can
tell.

fixes #58
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
kind/enhancement A net-new feature or improvement to an existing feature
Projects
None yet
Development

Successfully merging a pull request may close this issue.

1 participant