Skip to content

Conversation

@hawkadrian
Copy link
Contributor

What

Improves documentation for map_add_ons method in NodeBuilderWithComponents and WithLaunchContext by adding usage examples, purpose description, and cross-references to related types.

Why

The method serves as an escape hatch for accessing addon-specific configuration methods (like with_rpc_middleware) that don't have direct builder API equivalents. The previous minimal documentation made it unclear when and how to use this method, especially for advanced configuration scenarios.

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

makes sense

@github-project-automation github-project-automation bot moved this from Backlog to In Progress in Reth Tracker Dec 10, 2025
@mattsse mattsse enabled auto-merge December 10, 2025 08:41
auto-merge was automatically disabled December 10, 2025 10:01

Head branch was pushed to by a user without write access

@hawkadrian
Copy link
Contributor Author

@mattsse hey boss! there were lint docs errors, fixed it, now should be fine

Copy link
Collaborator

@mattsse mattsse left a comment

Choose a reason for hiding this comment

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

alright

@mattsse mattsse added this pull request to the merge queue Dec 11, 2025
Merged via the queue into paradigmxyz:main with commit 036626b Dec 11, 2025
44 checks passed
@github-project-automation github-project-automation bot moved this from In Progress to Done in Reth Tracker Dec 11, 2025
Vui-Chee added a commit to okx/reth that referenced this pull request Dec 12, 2025
…number

* upstream: (203 commits)
  feat(node-core): make rpc server args customizable (paradigmxyz#20312)
  feat: add `account_history_in_rocksdb` field to `StorageSettings` (paradigmxyz#20282)
  feat(engine): Add BAL stub methods to ExecutionPayload and BlockOrPayload (paradigmxyz#20311)
  docs: fix misleading links (paradigmxyz#20300)
  ci: add more sccache (paradigmxyz#20316)
  feat: bump alloy-evm (paradigmxyz#20314)
  feat: allow larger ws frames on client side (paradigmxyz#20307)
  docs: add architecture diagrams to ExEx documentation (paradigmxyz#20193)
  feat: add semaphore for blocking IO requests (paradigmxyz#20289)
  ci: scale down depot runners (paradigmxyz#20295)
  perf: fetch header directly (paradigmxyz#20294)
  docs(exex): fix DebugApi comment (paradigmxyz#20296)
  feat: add support for testing_ rpc namespace and testing_buildBlockV1 (paradigmxyz#20094)
  chore: update engine_getBlobs metric (paradigmxyz#20290)
  chore(optimism): move predeploy constant to op-alloy (paradigmxyz#20181)
  docs: fix stages order and add missing EraStage (paradigmxyz#20283)
  docs: improve map_add_ons method documentation (paradigmxyz#20248)
  feat: add `transaction_hash_numbers_in_rocksdb` field to `StorageSettings` (paradigmxyz#20209)
  docs: clarify network mode, tx gossip and NAT (paradigmxyz#20247)
  feat: add support for debug_getBadBlock (paradigmxyz#20177)
  ...
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants