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

Log l2 migration block in config #2341

Merged
merged 1 commit into from
Dec 5, 2024
Merged

Conversation

palango
Copy link
Contributor

@palango palango commented Dec 5, 2024

Description

Logs the L2 migration block.

@palango palango requested a review from gastonponti December 5, 2024 12:29
Copy link

github-actions bot commented Dec 5, 2024

Coverage from tests in ./e2e_test/... for ./consensus/istanbul/... at commit 1ebb1be

coverage: 55.2% of statements across all listed packages
coverage:  68.4% of statements in consensus/istanbul
coverage:  63.4% of statements in consensus/istanbul/announce
coverage:  57.1% of statements in consensus/istanbul/backend
coverage:   0.0% of statements in consensus/istanbul/backend/backendtest
coverage:  24.3% of statements in consensus/istanbul/backend/internal/replica
coverage:  66.3% of statements in consensus/istanbul/core
coverage:  50.0% of statements in consensus/istanbul/db
coverage:   0.0% of statements in consensus/istanbul/proxy
coverage:  64.2% of statements in consensus/istanbul/uptime
coverage:  52.4% of statements in consensus/istanbul/validator
coverage:  79.2% of statements in consensus/istanbul/validator/random

Copy link

github-actions bot commented Dec 5, 2024

5890 passed, 45 skipped

Copy link
Contributor

@gastonponti gastonponti left a comment

Choose a reason for hiding this comment

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

LGTM

@palango palango enabled auto-merge (squash) December 5, 2024 12:40
@palango palango merged commit 1ebb1be into master Dec 5, 2024
25 of 26 checks passed
@palango palango deleted the palango/config-display-l2 branch December 5, 2024 12:45
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.

2 participants