Skip to content

Conversation

@EgeCaner
Copy link
Contributor

No description provided.

@EgeCaner EgeCaner added the disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment. label Jan 15, 2026
@codecov
Copy link

codecov bot commented Jan 15, 2026

Codecov Report

❌ Patch coverage is 31.85484% with 338 lines in your changes missing coverage. Please review.
✅ Project coverage is 75.64%. Comparing base (7a3db92) to head (9b19628).

Files with missing lines Patch % Lines
migration/casmhashmetadata/migrator.go 6.45% 115 Missing and 1 partial ⚠️
migration/casmhashmetadata/ingestors.go 0.00% 113 Missing ⚠️
blockchain/blockchain.go 41.79% 35 Missing and 4 partials ⚠️
migration/casmhashmetadata/committer.go 0.00% 19 Missing ⚠️
migration/progresslogger/block_progress_tracker.go 0.00% 18 Missing ⚠️
core/state.go 42.85% 11 Missing and 1 partial ⚠️
migration/progresslogger/interval.go 0.00% 12 Missing ⚠️
core/class.go 91.91% 5 Missing and 3 partials ⚠️
migration/migration.go 83.33% 1 Missing ⚠️

❌ Your patch check has failed because the patch coverage (31.85%) is below the target coverage (60.00%). You can increase the patch coverage or adjust the target coverage.

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #3379      +/-   ##
==========================================
- Coverage   76.19%   75.64%   -0.56%     
==========================================
  Files         353      358       +5     
  Lines       33394    33848     +454     
==========================================
+ Hits        25444    25603     +159     
- Misses       6134     6428     +294     
- Partials     1816     1817       +1     

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

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

Copy link
Collaborator

@rodrodros rodrodros left a comment

Choose a reason for hiding this comment

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

Looks very good, just some easy commments

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

disable-deploy-test We don't want to run deploy tests with this PR because it might affect our development environment.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants