Skip to content

dependency: use the updated Nethermind branch#83

Merged
AnkushinDaniil merged 1 commit into
mainfrom
daniil/initialization-of-arbitrum-block-fix
Jul 8, 2025
Merged

dependency: use the updated Nethermind branch#83
AnkushinDaniil merged 1 commit into
mainfrom
daniil/initialization-of-arbitrum-block-fix

Conversation

@AnkushinDaniil

@AnkushinDaniil AnkushinDaniil commented Jul 7, 2025 •
edited
Loading

Copy link
Copy Markdown
Collaborator

Fixes #63

@AnkushinDaniil AnkushinDaniil changed the title dependancy: use the updated Nethermind branch with the fix dependency: use the updated Nethermind branch with the fix Jul 7, 2025
@codecov

codecov Bot commented Jul 7, 2025 •
edited
Loading

Copy link
Copy Markdown

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 84.18%. Comparing base (b47a806) to head (ec50a74).
Report is 1 commits behind head on main.

Additional details and impacted files 
@@           Coverage Diff           @@
##             main      #83   +/-   ##
=======================================
  Coverage   84.18%   84.18%           
=======================================
  Files          65       65           
  Lines        3269     3269           
  Branches      366      453   +87     
=======================================
  Hits         2752     2752           
  Misses        517      517
Flag Coverage Δ
unittests 84.18% <ø> (ø)

Continue to review full report in Codecov by Sentry.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update b47a806...ec50a74. Read the comment docs.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@AnkushinDaniil AnkushinDaniil force-pushed the daniil/initialization-of-arbitrum-block-fix branch from 4be23e2 to 15fef5a Compare July 8, 2025 07:41
@AnkushinDaniil AnkushinDaniil changed the title dependency: use the updated Nethermind branch with the fix dependency: use the updated Nethermind branch Jul 8, 2025
@AnkushinDaniil AnkushinDaniil force-pushed the daniil/initialization-of-arbitrum-block-fix branch from 15fef5a to ec50a74 Compare July 8, 2025 07:44
@AnkushinDaniil AnkushinDaniil marked this pull request as ready for review July 8, 2025 07:45
Copilot AI review requested due to automatic review settings July 8, 2025 07:45
Copilot AI reviewed Jul 8, 2025
View reviewed changes

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull Request Overview

Updates the Nethermind submodule to point at a newer commit.

  • Bumps the commit hash for the Nethermind dependency.

@AnkushinDaniil AnkushinDaniil merged commit a891bb5 into main Jul 8, 2025
8 checks passed
@AnkushinDaniil AnkushinDaniil deleted the daniil/initialization-of-arbitrum-block-fix branch July 8, 2025 09:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

Copilot code review Copilot Copilot left review comments

@wurdum wurdum wurdum approved these changes

@hudem1 hudem1 hudem1 approved these changes

@damian-orzechowski damian-orzechowski Awaiting requested review from damian-orzechowski damian-orzechowski is a code owner

@tanishqjasoria tanishqjasoria Awaiting requested review from tanishqjasoria

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

Initialization of Arbitrum block processing takes long time

4 participants

@AnkushinDaniil @wurdum @hudem1