Skip to content

feat: add a genesis configuration toml file#1000

Merged
bobbinth merged 81 commits intonextfrom
bernhard-toml-genesis
Jul 10, 2025
Merged

feat: add a genesis configuration toml file#1000
bobbinth merged 81 commits intonextfrom
bernhard-toml-genesis

Conversation

@drahnr
Copy link
Contributor

@drahnr drahnr commented Jun 23, 2025

Ref: #836
Superseeds: mmagician#1

@drahnr drahnr force-pushed the bernhard-toml-genesis branch from 26e24eb to 8af5e8c Compare June 24, 2025 09:48
@drahnr drahnr marked this pull request as ready for review June 26, 2025 14:39
@drahnr drahnr force-pushed the bernhard-toml-genesis branch 2 times, most recently from c831161 to f00946f Compare June 26, 2025 15:11
@drahnr drahnr force-pushed the bernhard-toml-genesis branch 2 times, most recently from 2744435 to 92d9d22 Compare June 30, 2025 12:00
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig left a comment

Choose a reason for hiding this comment

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

I think we still need to integrate this with the CLI of the node. That could also be a separate PR?

@Mirko-von-Leipzig Mirko-von-Leipzig self-requested a review June 30, 2025 12:25
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig left a comment

Choose a reason for hiding this comment

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

Didn't mean to approve just yet :)

@drahnr drahnr requested a review from Mirko-von-Leipzig June 30, 2025 17:19
Copy link
Collaborator

@Mirko-von-Leipzig Mirko-von-Leipzig left a comment

Choose a reason for hiding this comment

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

Some doc suggestions and questions :)

Copy link
Contributor

@mmagician mmagician left a comment

Choose a reason for hiding this comment

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

I'm looking forward to using this for integration testing, thanks. I left a few minor comments but functionality-wise looks good 👍🏼

@drahnr drahnr requested review from mmagician and sergerad July 3, 2025 16:37
Copy link
Collaborator

@sergerad sergerad left a comment

Choose a reason for hiding this comment

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

LGTM

@drahnr drahnr changed the title feat: add a test genesis configuration toml file feat: add a genesis configuration toml file Jul 4, 2025
drahnr and others added 7 commits July 9, 2025 10:54
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
Co-authored-by: Mirko <48352201+Mirko-von-Leipzig@users.noreply.github.com>
@drahnr drahnr force-pushed the bernhard-toml-genesis branch from 4298370 to 6c9ad51 Compare July 9, 2025 09:01
@drahnr drahnr force-pushed the bernhard-toml-genesis branch from 740891d to 4791086 Compare July 9, 2025 11:54
@drahnr
Copy link
Contributor Author

drahnr commented Jul 9, 2025

fn into_state has become a bit of a hydra, functionality wise - bar validation of validity of the final GenesisState ( see #1059 ) - all feedback is incorporated

Copy link
Contributor

@bobbinth bobbinth left a comment

Choose a reason for hiding this comment

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

All looks good! Thank you!

@bobbinth bobbinth merged commit 21dd5b4 into next Jul 10, 2025
8 checks passed
@bobbinth bobbinth deleted the bernhard-toml-genesis branch July 10, 2025 00:58
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.

6 participants