0xMiden / 0xMiden/node

Rename NetworkTransactionBuilder

Open
#1,620 0 comments 0 reactions 1 assignee Claimed by @SantiagoPittella View on GitHub
network transactions
Dominant language
Rust
Stars
104
Forks
138
Avg merge
1d 13h
Merged PRs (30d)
56

Description

Bit confusing with the stuttering builder/builder naming, but that's not this PRs fault. We should come up with something better than network transaction builder..

Maybe network transaction..
- producer
- service
- component (bleh)
- handler
- .. hm.

_Originally posted by @Mirko-von-Leipzig in https://github.com/0xMiden/miden-node/pull/1610#discussion_r2746178064_

The current naming of NetworkTransactionBuilder creates confusion with Rust's common "builder pattern" terminology.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.