Skip to content

fix TOO_MANY_STATE_MODIFICATIONS comment#768

Open
tequdev wants to merge 1 commit into
Xahau:devfrom
tequdev:fix-toomanystatemodifications-comment
Open

fix TOO_MANY_STATE_MODIFICATIONS comment#768
tequdev wants to merge 1 commit into
Xahau:devfrom
tequdev:fix-toomanystatemodifications-comment

Conversation

@tequdev

@tequdev tequdev commented Jul 10, 2026

Copy link
Copy Markdown
Member

High Level Overview of Change

Context of Change

Type of Change

  • Bug fix (non-breaking change which fixes an issue)

API Impact

  • Public API: New feature (new methods and/or new fields)
  • Public API: Breaking change (in general, breaking changes should only impact the next api_version)
  • libxrpl change (any change that may affect libxrpl or dependents of libxrpl)
  • Peer protocol change (must be backward compatible or bump the peer protocol version)

@codecov

codecov Bot commented Jul 10, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 77.3%. Comparing base (bb244ef) to head (2e86d29).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##             dev    #768     +/-   ##
=======================================
- Coverage   77.3%   77.3%   -0.0%     
=======================================
  Files        837     837             
  Lines      78585   78585             
  Branches   11559   11559             
=======================================
- Hits       60747   60735     -12     
- Misses     17828   17840     +12     
  Partials      10      10             
Files with missing lines Coverage Δ
include/xrpl/hook/Enum.h 93.3% <ø> (ø)

... and 10 files with indirect coverage changes

Impacted file tree graph

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

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.

1 participant