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

Remove nimiq-genesis as dependency for test-log #3266

Merged
merged 1 commit into from
Feb 5, 2025
Merged

Conversation

viquezclaudio
Copy link
Member

Fixes #2558

...

This fixes #2558 .

Pull request checklist

  • All tests pass. The project builds and runs.
  • I have resolved any merge conflicts.
  • I have resolved all clippy and rustfmt warnings.

@viquezclaudio viquezclaudio force-pushed the viquezcl/test-log branch 2 times, most recently from 2bdeb23 to b748185 Compare February 4, 2025 15:11
@sisou
Copy link
Member

sisou commented Feb 4, 2025

I think you need to remove nimiq-genesis also from the Cargo.toml to really prevent compilation when this crate is used.

@@ -659,7 +659,8 @@ pub const TEST_POLICY: Policy = Policy {
batches_per_epoch: 4,
state_chunks_max_size: 3,
transaction_validity_window: 2,
genesis_block_number: 0,
// This number should match the one that is defined in the unittest genesis file
Copy link
Member

@jsdanielh jsdanielh Feb 4, 2025

Choose a reason for hiding this comment

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

Suggested change
// This number should match the one that is defined in the unittest genesis file
// This number should match the one that is defined in the `unit` network genesis file which is the genesis used for unit testing

Copy link
Contributor

@hrxi hrxi left a comment

Choose a reason for hiding this comment

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

Amazing. This dependency has been a thorn in my side for a long time already.

@hrxi hrxi force-pushed the viquezcl/test-log branch from 562d7de to ea5f678 Compare February 5, 2025 08:55
@hrxi
Copy link
Contributor

hrxi commented Feb 5, 2025

Rebased for merging.

@hrxi hrxi merged commit ea5f678 into albatross Feb 5, 2025
8 checks passed
@hrxi hrxi deleted the viquezcl/test-log branch February 5, 2025 09:28
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.

nimiq-test-log should not transitively depend on nimiq-genesis
4 participants