Skip to content

Conversation

@smelc
Copy link
Contributor

@smelc smelc commented Mar 17, 2025

Description

While working on other issues, I noticed that the enable P2P option was not implemented, so this PR removes it.

This is in line with #6148 and #6137 that move towards implementing such flags using the node's configuration file, instead of having dedicated flags.

Checklist

  • Commit sequence broadly makes sense and commits have useful messages
  • New tests are added if needed and existing tests are updated. These may include:
    • golden tests
  • CI passes. See note on CI. The following CI checks are required:
  • Self-reviewed the diff

@smelc smelc marked this pull request as ready for review March 17, 2025 20:04
@smelc smelc requested a review from a team as a code owner March 17, 2025 20:04
@smelc smelc force-pushed the smelc/cardano-testnet-remove-unused-option branch 2 times, most recently from f8ad07e to 5a2bd94 Compare March 18, 2025 09:37
@smelc smelc enabled auto-merge March 19, 2025 08:44
@carbolymer carbolymer disabled auto-merge March 19, 2025 15:21
@carbolymer carbolymer enabled auto-merge March 19, 2025 15:21
@smelc smelc force-pushed the smelc/cardano-testnet-remove-unused-option branch from 5a2bd94 to cecdf60 Compare March 21, 2025 09:09
@carbolymer carbolymer added this pull request to the merge queue Mar 21, 2025
Merged via the queue into master with commit 9932726 Mar 21, 2025
23 checks passed
@carbolymer carbolymer deleted the smelc/cardano-testnet-remove-unused-option branch March 21, 2025 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants