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: update faucets #6024

Conversation

hansieodendaal
Copy link
Contributor

@hansieodendaal hansieodendaal commented Dec 8, 2023

Description

Updated the faucets and initial emission values for esmeralda, stagenet and nextnet.

Note: The igor faucet has been uploaded but not activated.

Motivation and Context

Faucet values were not accounted for in the total emission.

How Has This Been Tested?

Existing unit tests pass.
Updated unit test fn esmeralda_schedule().
Updated the esmeralda, stagenet and nextnet genesis blocks with new faucet MMR information.

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

Code walk-through
Run the unit test

Breaking Changes

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

BREAKING CHANGE: The block chain needs to be started from scratch

@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 Dec 8, 2023
Update the faucets and initial emission values for esmeralda, igor, stagenet and nextnet.
@ghpbot-tari-project ghpbot-tari-project removed the P-reviews_required Process - Requires a review from a lead maintainer to be merged label Dec 8, 2023
Copy link

github-actions bot commented Dec 8, 2023

Test Results (CI)

586 tests   585 ✔️  8m 29s ⏱️
  27 suites      0 💤
    1 files        1

For more details on these failures, see this check.

Results for commit 21fd26a.

@SWvheerden SWvheerden merged commit 394976c into tari-project:development Dec 8, 2023
10 of 13 checks passed
SWvheerden pushed a commit that referenced this pull request Dec 9, 2023
Description
---
Added more faucet tests, updated constants

Motivation and Context
---
This should have been part of #6024

How Has This Been Tested?
---
New unit tests

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

<!-- Checklist -->
<!-- 1. Is the title of your PR in the form that would make nice release
notes? The title, excluding the conventional commit
tag, will be included exactly as is in the CHANGELOG, so please think
about it carefully. -->


Breaking Changes
---

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

<!-- Does this include a breaking change? If so, include this line as a
footer -->
<!-- BREAKING CHANGE: Description what the user should do, e.g. delete a
database, resync the chain -->
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.

3 participants