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: trickle validators on layer 1 #1197

Merged

Conversation

ksrichard
Copy link
Collaborator

Description

Motivation and Context

How Has This Been Tested?

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

Breaking Changes

  • None
  • Requires data directory to be deleted
  • Other - Please specify

@ksrichard ksrichard changed the title WIP -feat: trickle validators on layer 1 WIP: feat: trickle validators on layer 1 Nov 5, 2024
Copy link

github-actions bot commented Nov 5, 2024

Test Results (CI)

561 tests  ±0   561 ✅ ±0   1h 25m 20s ⏱️ + 1m 18s
 54 suites ±0     0 💤 ±0 
  1 files   ±0     0 ❌ ±0 

Results for commit fe1efab. ± Comparison against base commit a148dcb.

♻️ This comment has been updated with latest results.

@ksrichard ksrichard changed the title WIP: feat: trickle validators on layer 1 feat: trickle validators on layer 1 Nov 8, 2024
@ksrichard ksrichard marked this pull request as ready for review November 8, 2024 13:43
sdbondi
sdbondi previously approved these changes Nov 11, 2024
Copy link
Member

@sdbondi sdbondi left a comment

Choose a reason for hiding this comment

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

LGTM - tested with 5 initial and 1 per epoch and works well

sdbondi
sdbondi previously approved these changes Nov 11, 2024
@sdbondi sdbondi enabled auto-merge November 11, 2024 13:10
@ksrichard ksrichard disabled auto-merge November 11, 2024 13:14
sdbondi
sdbondi previously approved these changes Nov 11, 2024
@sdbondi sdbondi disabled auto-merge November 12, 2024 07:55
@sdbondi sdbondi merged commit 9907bc1 into tari-project:development Nov 12, 2024
10 of 11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants