Skip to content

feat(l1) add holesky presets#1601

Merged
fmoletta merged 7 commits into
mainfrom
add-holesky-presets
Jan 8, 2025
Merged

feat(l1) add holesky presets#1601
fmoletta merged 7 commits into
mainfrom
add-holesky-presets

Conversation

@fmoletta

@fmoletta fmoletta commented Jan 2, 2025

Copy link
Copy Markdown
Contributor

Motivation

Description

Closes #1596

@@ -0,0 +1,1015 @@
{

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nit: maybe move to networks/holesky/genesis.json?

@fmoletta fmoletta marked this pull request as ready for review January 8, 2025 13:30
@fmoletta fmoletta requested a review from a team as a code owner January 8, 2025 13:30
Comment thread cmd/ethrex/holesky_presets.rs Outdated

pub const HOLESKY_GENESIS_PATH: &str = "cmd/ethrex/networks/holesky/genesis.json";

lazy_static! {

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

maybe we can move the bootnodes to networks/holesky/bootnodes.json?

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

I want that just adding a new network means adding a couple of json files in networks/{network_name}

@github-actions

github-actions Bot commented Jan 8, 2025

Copy link
Copy Markdown
+-------------------------------------------------------------------+-------+------+
| File                                                              | Lines | Diff |
+-------------------------------------------------------------------+-------+------+
| /home/runner/work/ethrex/ethrex/cmd/ethrex/ethrex.rs              | 334   | +7   |
+-------------------------------------------------------------------+-------+------+
| /home/runner/work/ethrex/ethrex/cmd/ethrex/networks.rs            | 10    | +10  |
+-------------------------------------------------------------------+-------+------+
| /home/runner/work/ethrex/ethrex/crates/networking/p2p/bootnode.rs | 48    | +8   |
+-------------------------------------------------------------------+-------+------+

Total lines changed: +25

@mpaulucci mpaulucci left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

nice!

@fmoletta fmoletta added this pull request to the merge queue Jan 8, 2025
Merged via the queue into main with commit 454f809 Jan 8, 2025
@fmoletta fmoletta deleted the add-holesky-presets branch January 8, 2025 17:57
pedrobergamini pushed a commit to pedrobergamini/ethrex that referenced this pull request Aug 24, 2025
**Motivation**

<!-- Why does this pull request exist? What are its goals? -->

**Description**

<!-- A clear and concise general description of the changes this PR
introduces -->

<!-- Link to issues: Resolves lambdaclass#111, Resolves lambdaclass#222 -->

Closes lambdaclass#1596
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.

Add configurations for Holesky testnet

2 participants