Skip to content

Conversation

@JekaMas
Copy link
Contributor

@JekaMas JekaMas commented Dec 23, 2025

  1. Remove subscriber in a few places,
  2. Generalizes an access map

@github-actions
Copy link

Code Metrics Report

#32 (cd19024) #33 (193b734) +/-
Coverage 48.2% 51.9% +3.7%
Test Execution Time 1m10s 1m10s 0s
Details
  |                     | #32 (cd19024) | #33 (193b734) |  +/-  |
  |---------------------|---------------|---------------|-------|
+ | Coverage            |         48.2% |         51.9% | +3.7% |
  |   Files             |            38 |            37 |    -1 |
  |   Lines             |          2639 |          2553 |   -86 |
+ |   Covered           |          1273 |          1327 |   +54 |
  | Test Execution Time |         1m10s |         1m10s |    0s |

Code coverage of files in pull request scope (27.1% → 36.9%)

Files Coverage +/- Status
gosdk/appchain.go 34.8% +0.2% modified
gosdk/evmtypes/header.go 82.2% +2.2% affected
gosdk/init.go 0.0% -3.5% affected
gosdk/mdbx_event_wrapper.go 0.0% 0.0% modified
gosdk/multichain.go 75.0% -0.9% modified
gosdk/multichain_sql.go 61.4% +61.4% modified
gosdk/state_transition.go 2.2% -0.1% affected

Reported by octocov

@0xaigen 0xaigen merged commit 13d1437 into main Dec 24, 2025
1 check passed
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.

3 participants