Conversation
…d go-perun v0.15 Signed-off-by: Minh Huy Tran <huy@perun.network>
Signed-off-by: Minh Huy Tran <huy@perun.network>
…hardhat node Signed-off-by: Minh Huy Tran <huy@perun.network>
…m hardhat devnet Signed-off-by: Minh Huy Tran <huy@perun.network>
Signed-off-by: Minh Huy Tran <huy@perun.network>
Signed-off-by: Minh Huy Tran <huy@perun.network>
Signed-off-by: Minh Huy Tran <huy@perun.network>
Signed-off-by: Minh Huy Tran <huy@perun.network>
There was a problem hiding this comment.
Pull request overview
This PR updates the go-perun examples to version 0.15.0 and migrates the Ethereum development environment from Ganache to Hardhat. The changes consolidate peer-to-peer networking setup by integrating libp2p directly into go-perun and replacing the external perun-libp2p-wire dependency.
Changes:
- Updated go-perun dependency from v0.13.1 to v0.15.0 across all example projects
- Migrated Ethereum local node setup from Ganache CLI to Hardhat development node
- Refactored P2P bus setup to use integrated libp2p implementation from go-perun
Reviewed changes
Copilot reviewed 50 out of 57 changed files in this pull request and generated no comments.
Show a summary per file
| File | Description |
|---|---|
| payment-channel/util.go | Added setupBusWire helper function for P2P networking |
| payment-channel/main.go | Refactored to use new setupBusWire function instead of external libp2p library |
| payment-channel/package.json | Added Hardhat as development dependency |
| payment-channel/hardhat.config.js | Configured Hardhat node with test accounts and mining settings |
| payment-channel/go.mod | Updated go-perun to v0.15.0, removed perun-libp2p-wire dependency |
| payment-channel/README.md | Updated documentation to reflect Hardhat usage |
| .github/workflows/ci.yml | Updated CI workflow to use Hardhat instead of Ganache |
| multiledger-channel/. | Applied same migration pattern for multiledger examples |
| app-channel/. | Applied same migration pattern for app channel examples |
| payment-channel-xlm/. | Applied same migration pattern for Stellar examples |
| payment-channel-icp/. | Applied same migration pattern for ICP examples |
| payment-channel-dot/. | Applied same migration pattern for Polkadot examples |
| payment-channel-ckb-eth/. | Applied same migration pattern for CKB-ETH examples |
| payment-channel-cc/. | Applied same migration pattern for cross-contract examples |
Comments suppressed due to low confidence (2)
payment-channel/hardhat.config.js:1
- The 'type' field is not a standard Hardhat configuration option. If this is a custom configuration for a specific plugin or feature, it should be documented in comments explaining its purpose.
payment-channel/util.go:1 - The function name 'setupBusWire' is ambiguous. Consider renaming to 'setupP2PBus' or 'createP2PBusWithDialer' to better reflect that it creates a bus instance and returns a dialer.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
iljabvh
approved these changes
Mar 2, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
This PR aims to solve the issue #50. It should update the examples to go-perun v0.15.0 and update the ethereum CI to migrate from Ganache to Hardhat developement node.