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

feat: add esmeralda pre-mine #6454

Merged

Conversation

hansieodendaal
Copy link
Contributor

@hansieodendaal hansieodendaal commented Aug 6, 2024

Description

Added pre-mine information to the esmeralda genesis block

Note: Temporarily reverted change introduced by release v1.1.0-pre.2, will be fixed in #6452.

Motivation and Context

Testing pre-mine

How Has This Been Tested?

Unit tests

What process can a PR reviewer use to test or verify this change?

Code review

Breaking Changes

  • None
  • Requires data directory on base node to be deleted
  • Requires hard fork
  • Other - Please specify

@hansieodendaal hansieodendaal requested a review from a team as a code owner August 6, 2024 10:20
Copy link

github-actions bot commented Aug 6, 2024

Test Results (CI)

    3 files    123 suites   39m 58s ⏱️
1 300 tests 1 300 ✅ 0 💤 0 ❌
3 892 runs  3 892 ✅ 0 💤 0 ❌

Results for commit e08cf4c.

♻️ This comment has been updated with latest results.

@ghpbot-tari-project ghpbot-tari-project added P-acks_required Process - Requires more ACKs or utACKs P-reviews_required Process - Requires a review from a lead maintainer to be merged labels Aug 6, 2024
Copy link

github-actions bot commented Aug 6, 2024

Test Results (Integration tests)

19 tests   19 ✅  11m 19s ⏱️
 9 suites   0 💤
 2 files     0 ❌
 1 errors

For more details on these parsing errors, see this check.

Results for commit e08cf4c.

♻️ This comment has been updated with latest results.

stringhandler
stringhandler previously approved these changes Aug 6, 2024
SWvheerden
SWvheerden previously approved these changes Aug 6, 2024
@hansieodendaal hansieodendaal dismissed stale reviews from SWvheerden and stringhandler via 92f4dcb August 6, 2024 11:49
Added pre-mine information to the esmeralda genesis block
@hansieodendaal hansieodendaal force-pushed the ho_esmeralda_genesis_block branch from 5d94377 to e08cf4c Compare August 6, 2024 14:26
@ghpbot-tari-project ghpbot-tari-project removed the P-reviews_required Process - Requires a review from a lead maintainer to be merged label Aug 6, 2024
@SWvheerden SWvheerden merged commit c38a948 into tari-project:development Aug 6, 2024
15 of 17 checks passed
@hansieodendaal hansieodendaal deleted the ho_esmeralda_genesis_block branch August 7, 2024 07:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
P-acks_required Process - Requires more ACKs or utACKs
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants