Skip to content

Conversation

kselveliev
Copy link
Contributor

@kselveliev kselveliev commented Jul 16, 2025

Description:
This PR enables contract slot caching after regression fixes

Checklist

  • Documented (Code comments, README, etc.)
  • Tested (unit, integration, etc.)

Signed-off-by: Kristiyan Selveliev <[email protected]>
@kselveliev kselveliev self-assigned this Jul 16, 2025
@kselveliev kselveliev requested a review from a team as a code owner July 16, 2025 07:03
@kselveliev kselveliev added enhancement Type: New feature web3 Area: Web3 API labels Jul 16, 2025
@kselveliev kselveliev requested a review from filev94 July 16, 2025 07:03
@lfdt-bot
Copy link

🎉 Snyk checks have passed. No issues have been found so far.

security/snyk check is complete. No issues have been found. (View Details)

Copy link

Coverage summary from Codacy

See diff coverage on Codacy

Coverage variation Diff coverage
-32.81% (target: -1.00%)
Coverage variation details
Coverable lines Covered lines Coverage
Common ancestor commit (4a711e9) 36149 33927 93.85%
Head commit (5545f1d) 59695 (+23546) 36437 (+2510) 61.04% (-32.81%)

Coverage variation is the difference between the coverage for the head and common ancestor commits of the pull request branch: <coverage of head commit> - <coverage of common ancestor commit>

Diff coverage details
Coverable lines Covered lines Diff coverage
Pull request (#11592) 0 0 ∅ (not applicable)

Diff coverage is the percentage of lines that are covered by tests out of the coverable lines that the pull request added or modified: <covered lines added or modified>/<coverable lines added or modified> * 100%

See your quality gate settings    Change summary preferences

Copy link
Contributor

@xin-hedera xin-hedera left a comment

Choose a reason for hiding this comment

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

LGTM

@steven-sheehy steven-sheehy merged commit b0e9f00 into main Jul 16, 2025
32 of 33 checks passed
@steven-sheehy steven-sheehy deleted the 11551-enable-slot-caching branch July 16, 2025 19:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Type: New feature web3 Area: Web3 API
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants