Skip to content
This repository has been archived by the owner on Jun 20, 2023. It is now read-only.

Version 0.5.0 #31

Merged
merged 2 commits into from
Apr 27, 2022
Merged

Version 0.5.0 #31

merged 2 commits into from
Apr 27, 2022

Conversation

guseggert
Copy link
Contributor

This includes new OpenTelemetry dependencies and the addition of tracing spans.

@BigLep
Copy link

BigLep commented Apr 26, 2022

2022-04-26 conversation: need to bump version numbers and then will merge.

Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@guseggert before you make a new release, mind reviewing and merging fix from #32?
(fixes regression with go-libp2p 0.19)

Copy link
Member

@lidel lidel left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM (rebased to include #31).

@guseggert mind merging, so we get a release that I can switch to in ipfs/kubo#8868?

@github-actions
Copy link

Suggested version: v0.5.0
Comparing to: v0.4.0 (diff)

Changes in go.mod file(s):

diff --git a/go.mod b/go.mod
index 9588e01..b867274 100644
--- a/go.mod
+++ b/go.mod
@@ -24,6 +24,8 @@ require (
 	github.com/multiformats/go-multiaddr-dns v0.3.1
 	github.com/multiformats/go-multihash v0.0.15
 	github.com/whyrusleeping/base32 v0.0.0-20170828182744-c30ac30633cc
+	go.opentelemetry.io/otel v1.6.3
+	go.opentelemetry.io/otel/trace v1.6.3
 )
 
 go 1.16

gorelease says:

# github.com/ipfs/go-namesys
## compatible changes
StartSpan: added

# summary
Suggested version: v0.5.0

gocompat says:

(empty)

@guseggert guseggert merged commit ba24a61 into master Apr 27, 2022
@guseggert guseggert deleted the guseggert-patch-1 branch April 27, 2022 23:33
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
No open projects
Archived in project
Development

Successfully merging this pull request may close these issues.

4 participants