Skip to content

Conversation

@rvagg
Copy link
Member

@rvagg rvagg commented Oct 11, 2025

Includes minor fix for their migration away from ipfs/go-log: libp2p/go-libp2p#3364

@rvagg rvagg requested review from Copilot and rjan90 October 11, 2025 05:11
@github-project-automation github-project-automation bot moved this to 📌 Triage in FilOz Oct 11, 2025
Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This pull request upgrades libp2p dependencies from version 0.43 to 0.44 and makes necessary code adjustments for API compatibility. The primary motivation is to migrate away from the deprecated ipfs/go-log package as described in the referenced libp2p migration.

  • Upgraded libp2p core library from v0.43.0 to v0.44.0
  • Updated pubsub library from v0.13.0 to v0.15.0
  • Added gologshim logger parameter to NewRemoteTracer call

Reviewed Changes

Copilot reviewed 2 out of 3 changed files in this pull request and generated no comments.

File Description
go.mod Updates libp2p and related dependency versions to 0.44.x series
node/modules/lp2p/pubsub.go Adds gologshim import and logger parameter for API compatibility

Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.

@rvagg rvagg added the skip/changelog This change does not require CHANGELOG.md update label Oct 11, 2025
@rvagg rvagg enabled auto-merge (rebase) October 11, 2025 07:04
@rjan90 rjan90 moved this from 📌 Triage to 🔎 Awaiting Review in FilOz Oct 11, 2025
@rvagg rvagg merged commit 273b068 into master Oct 11, 2025
99 checks passed
@rvagg rvagg deleted the rvagg/libp2p branch October 11, 2025 07:24
@github-project-automation github-project-automation bot moved this from 🔎 Awaiting Review to 🎉 Done in FilOz Oct 11, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

skip/changelog This change does not require CHANGELOG.md update

Projects

Status: 🎉 Done

Development

Successfully merging this pull request may close these issues.

3 participants