Sponsored Content
Skip to content

Retired fix1543 amendment - #5926

Merged
bthomee merged 1 commit into
developfrom
pratik/Retire_fix1543_amendment
Oct 28, 2025
Merged

Retired fix1543 amendment#5926
bthomee merged 1 commit into
developfrom
pratik/Retire_fix1543_amendment

Conversation

@pratikmankawde

Copy link
Copy Markdown
Contributor

High Level Overview of Change

This change retires the fix1543 amendment.

Context of Change

Amendments activated for more than 2 years can be retired.
https://xrpl.org/resources/known-amendments#fix1543

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

@pratikmankawde
pratikmankawde requested review from a team and bthomee October 23, 2025 13:26
@bthomee
bthomee requested a review from a1q123456 October 23, 2025 13:36
@codecov

codecov Bot commented Oct 23, 2025

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 78.3%. Comparing base (76f774e) to head (1ff62d0).
⚠️ Report is 2 commits behind head on develop.

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##           develop   #5926   +/-   ##
=======================================
  Coverage     78.3%   78.3%           
=======================================
  Files          817     817           
  Lines        68991   68981   -10     
  Branches      8326    8324    -2     
=======================================
- Hits         54022   54019    -3     
+ Misses       14969   14962    -7     
Files with missing lines Coverage Δ
src/xrpld/app/tx/detail/Escrow.cpp 99.2% <ø> (-<0.1%) ⬇️
src/xrpld/app/tx/detail/Escrow.h 100.0% <ø> (ø)
src/xrpld/app/tx/detail/PayChan.cpp 96.3% <100.0%> (-0.1%) ⬇️
src/xrpld/app/tx/detail/PayChan.h 100.0% <ø> (ø)

... and 3 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.

@pratikmankawde pratikmankawde added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Oct 23, 2025
@bthomee bthomee removed the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Oct 23, 2025
@bthomee

bthomee commented Oct 23, 2025

Copy link
Copy Markdown
Collaborator

@pratikmankawde there's a conflict - can you please resolve it first? I'd also like to get a sign-off from @a1q123456 before merging.

Comment thread src/xrpld/app/tx/detail/Escrow.cpp Outdated
Comment thread src/xrpld/app/tx/detail/PayChan.cpp Outdated
Comment thread src/xrpld/app/tx/detail/Escrow.cpp Outdated
Comment thread src/xrpld/app/tx/detail/Escrow.cpp Outdated
@pratikmankawde

Copy link
Copy Markdown
Contributor Author

@pratikmankawde there's a conflict - can you please resolve it first? I'd also like to get a sign-off from @a1q123456 before merging.

Done

@bthomee

bthomee commented Oct 24, 2025

Copy link
Copy Markdown
Collaborator

@pratikmankawde There are some linking errors due to removing some of the getFlagsMask overrides - I'm sure it's an easy fix.

@bthomee
bthomee requested a review from a1q123456 October 24, 2025 14:51

@a1q123456 a1q123456 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.

lgtm

@bthomee

bthomee commented Oct 28, 2025

Copy link
Copy Markdown
Collaborator

@pratikmankawde Ready to merge?

@pratikmankawde pratikmankawde added the Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required. label Oct 28, 2025
@pratikmankawde

Copy link
Copy Markdown
Contributor Author

@pratikmankawde Ready to merge?

Yeah.

Amendments activated for more than 2 years can be retired. This change retires the fix1543 amendment.
@bthomee
bthomee force-pushed the pratik/Retire_fix1543_amendment branch from 11a99ab to 1ff62d0 Compare October 28, 2025 14:46
@bthomee
bthomee enabled auto-merge October 28, 2025 14:47
@bthomee
bthomee added this pull request to the merge queue Oct 28, 2025
Merged via the queue into develop with commit d9960d5 Oct 28, 2025
3 checks passed
@bthomee
bthomee deleted the pratik/Retire_fix1543_amendment branch October 28, 2025 15:40
@mvadari mvadari added this to the 3.2.0 milestone May 20, 2026
beartec-jpg pushed a commit to beartec-jpg/FalconLedger that referenced this pull request Jun 1, 2026
Amendments activated for more than 2 years can be retired. This change retires the fix1543 amendment.

Co-authored-by: Bart Thomee <11445373+bthomee@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Ready to merge *PR author* thinks it's ready to merge. Has passed code review. Perf sign-off may still be required.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants