Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Removed NonP2P code #5007

Open
wants to merge 5 commits into
base: main
Choose a base branch
from
Open

Removed NonP2P code #5007

wants to merge 5 commits into from

Conversation

coot
Copy link
Contributor

@coot coot commented Nov 4, 2024

Description

  • Removed NonP2P stack.
  • Renamed varous modules for better consistency.

Checklist

Quality

  • Commit sequence makes sense and have useful messages, see ref.
  • New tests are added and existing tests are updated.
  • Self-reviewed the PR.

Maintenance

  • Linked an issue or added the PR to the current sprint of ouroboros-network project.
  • Added labels.
  • Updated changelog files.
  • The documentation has been properly updated, see ref.

@coot coot requested a review from a team as a code owner November 4, 2024 11:07
@coot coot marked this pull request as draft November 4, 2024 11:10
@coot coot force-pushed the coot/drop-non-p2p branch from b964eba to b0d5f13 Compare November 4, 2024 11:55
@coot coot marked this pull request as ready for review November 4, 2024 12:44
@coot coot force-pushed the coot/drop-non-p2p branch from b0d5f13 to e6a376d Compare November 4, 2024 12:48
@coot coot added the diffusion Issues / PRs related to diffusion layer label Nov 4, 2024
@coot coot self-assigned this Nov 4, 2024
@coot coot force-pushed the coot/drop-non-p2p branch 3 times, most recently from 61518a8 to 8cad967 Compare December 19, 2024 06:04
coot added 5 commits January 3, 2025 09:01
* `Ouroboros.Network.Subscription` removed;
* `Ouroboros.Network.ErrorPolicy` removed;
* APIs removed from `Ouroboros.Network.Socket`:
  * `NetworkMutableState` & friends,
  * `withServerNode` and `withServerNode'`,
  * `NetworkServerTracers`,
  * `fromSnocket`,
  * `beginConnection`
* `Ouroboros.Network.Server.Socket` replaced with a simpler server
  implementation in `Test.Ouroboros.Network.Server`.

All tests & demos of `ouroboros-network-framework` update.
* Updated to compile with changes in the previous commit.
* APIs removed from `Ouroboros.Network.{NodeToClient,NodeToNode}` modules:
  * NetworkServerTracers
  * NetworkMutableState APIs
  * withServer
  * ErrorPolicies
  * WithAddr
  * SuspendDecision
* APIs removed from `Ouroboros.Network.NodeToNode` module:
  * IPSubscriptionTarget
  * NetworkIPSubscription
  * NetworkSubscriptionTracers
  * SubscriptionParams
  * DnsSubscriptionTarget
  * DnsSubscriptioinParams
  * NetworkDNSSubscriptionTracers
  * dnsSubscriptionWorker
@coot coot force-pushed the coot/drop-non-p2p branch from 8cad967 to 9ed2049 Compare January 3, 2025 08:08
@coot coot added PI-9 Issues / PRs related to PI-9 technical debt labels Jan 24, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
diffusion Issues / PRs related to diffusion layer PI-9 Issues / PRs related to PI-9 technical debt
Projects
Status: In Progress
Development

Successfully merging this pull request may close these issues.

1 participant