Skip to content

chore(feature_activation): update start_height before release#1198

Merged
jansegre merged 1 commit intomasterfrom
chore/update-feature-activation-test
Jan 9, 2025
Merged

chore(feature_activation): update start_height before release#1198
jansegre merged 1 commit intomasterfrom
chore/update-feature-activation-test

Conversation

@jansegre
Copy link
Member

@jansegre jansegre commented Jan 6, 2025

Motivation

The original test window for the transaction feature activation on testnet has passed.

Acceptance Criteria

  • Move the time window to the present.

Checklist

  • If you are requesting a merge into master, confirm this code is production-ready and can be included in future releases as soon as it gets merged

@jansegre jansegre self-assigned this Jan 6, 2025
@jansegre jansegre requested a review from msbrogli as a code owner January 6, 2025 14:06
@github-actions
Copy link

github-actions bot commented Jan 6, 2025

🐰 Bencher Report

Branchchore/update-feature-activation-test
Testbedubuntu-22.04
Click to view all benchmark results
BenchmarkLatencyBenchmark Result
minutes (m)
(Result Δ%)
Lower Boundary
minutes (m)
(Limit %)
Upper Boundary
minutes (m)
(Limit %)
sync-v2 (up to 20000 blocks)📈 view plot
🚷 view threshold
1.54
(-6.84%)
1.49
(96.61%)
1.82
(84.69%)
🐰 View full continuous benchmarking report in Bencher

msbrogli
msbrogli previously approved these changes Jan 6, 2025
@jansegre jansegre force-pushed the chore/update-feature-activation-test branch from 5ae35e6 to d120c61 Compare January 6, 2025 18:52
@jansegre jansegre requested a review from luislhl January 8, 2025 14:34
@jansegre jansegre force-pushed the chore/update-feature-activation-test branch from d120c61 to a538778 Compare January 8, 2025 14:34
@codecov
Copy link

codecov bot commented Jan 8, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.88%. Comparing base (b17a8be) to head (a538778).
Report is 2 commits behind head on master.

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1198      +/-   ##
==========================================
- Coverage   85.01%   84.88%   -0.14%     
==========================================
  Files         323      323              
  Lines       25124    25124              
  Branches     3859     3859              
==========================================
- Hits        21359    21326      -33     
- Misses       3027     3050      +23     
- Partials      738      748      +10     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@jansegre jansegre merged commit ae6a85c into master Jan 9, 2025
@jansegre jansegre deleted the chore/update-feature-activation-test branch January 9, 2025 16:34
This was referenced Jan 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

3 participants